Skip to content

[PWGHF] configurable name changed#16210

Open
sejeong8 wants to merge 1 commit into
AliceO2Group:masterfrom
sejeong8:namechanges
Open

[PWGHF] configurable name changed#16210
sejeong8 wants to merge 1 commit into
AliceO2Group:masterfrom
sejeong8:namechanges

Conversation

@sejeong8
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

O2 linter results: ❌ 0 errors, ⚠️ 0 warnings, 🔕 0 disabled

@github-actions github-actions Bot changed the title configurable name changed [PWGHF] configurable name changed May 12, 2026
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented May 12, 2026

configurable name changed where?

@sejeong8
Copy link
Copy Markdown
Contributor Author

configurable name changed where? : configurable and histogram name.

nCrossedRowTPCMin -> nCrossedRowTpcMin
Chi2pernClTpcMax -> chi2perNClTpcMax
Chi2pernClItsMax ->chi2perNClItsMax
ClsIBItsMin -> clsIbItsMin
nSigmaTPCMax -> nSigmaTpcMax
hTPCNSig -> hTpcNSig
hibClsItsTrack -> hIbClsItsTrack
hDcaChram -> hDcaCharm

@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented May 13, 2026

configurable name changed where? : configurable and histogram name.

nCrossedRowTPCMin -> nCrossedRowTpcMin Chi2pernClTpcMax -> chi2perNClTpcMax Chi2pernClItsMax ->chi2perNClItsMax ClsIBItsMin -> clsIbItsMin nSigmaTPCMax -> nSigmaTpcMax hTPCNSig -> hTpcNSig hibClsItsTrack -> hIbClsItsTrack hDcaChram -> hDcaCharm

This is not what I meant. You are supposed to summarise what changes and where in the PR title and where means file or analysis or feature... There are hundreds of configurables in PWGHF so the current title is very ambiguous and vague.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pwghf PWG-HF

Development

Successfully merging this pull request may close these issues.

2 participants