Skip to content

[PWGJE] Small incorrect code for histogram name and name of table #12925

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
nzardosh merged 17 commits into from
Sep 12, 2025
Merged

[PWGJE] Small incorrect code for histogram name and name of table #12925

nzardosh merged 17 commits into from
Sep 12, 2025

Conversation

@hanseopark
Copy link
Contributor

@hanseopark hanseopark commented Sep 9, 2025 •
edited
Loading

  • Changing histogram name and reduce axis
  • fixing table's name in process function
    in PWGJE/Tasks/jettaggerhfQA.cxx

hanseopark added 17 commits March 7, 2025 22:58
@hanseopark
add validation function and small fix
52394f1
@hanseopark
Merge branch 'master' into HFTagging
d96048e
@hanseopark
fix axis name
f542f10
@hanseopark
Merge branch 'master' into HFTagging
ca85f44
@hanseopark
update secondary contamination inside jets
059be8e
@hanseopark
Merge branch 'master' into HFTagging
59f03f1
@hanseopark
add secondary track inside jet when comapred to data with mc for syst…
b9e68df 
…ematic uncertainty
@hanseopark
Merge branch 'master' into HFTagging
6c8aabb
@hanseopark
fix clang-format and o2-linter
edb921f
@hanseopark
change int to std::size_t
93b1062
@hanseopark
update issue of o2-linter ofr task name
111dae5
@hanseopark
Merge branch 'master' into HFTagging
f44f588
@hanseopark
reduce member of struct because of limit under 100 and add o2-linter …
082f540 
…option again
@hanseopark
change comment for o2-linter
94fbaf6
@hanseopark
change the name of const variable and move them in function
f4391ee
@hanseopark
Merge branch 'master' into HFTagging
1333acc
@hanseopark
fixing the incorrect code
c4556a1
@github-actions github-actions bot added the pwgje label Sep 9, 2025
@github-actions
Copy link

github-actions bot commented Sep 9, 2025

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

@github-actions github-actions bot changed the title Fixsing incorrect code [PWGJE] Fixsing incorrect code Sep 9, 2025
@vkucera
Copy link
Collaborator

vkucera commented Sep 9, 2025

Please make your PR title more informative. The current one does not say anything about what was fixed and where.

@hanseopark hanseopark changed the title [PWGJE] Fixsing incorrect code [PWGJE] Small incorrect code for histogram name and name of table Sep 9, 2025
@hanseopark
Copy link
Contributor Author

hanseopark commented Sep 9, 2025

Please make your PR title more informative. The current one does not say anything about what was fixed and where.

I changed title and description

@vkucera
Copy link
Collaborator

vkucera commented Sep 9, 2025

Please make your PR title more informative. The current one does not say anything about what was fixed and where.

I changed title and description

Thanks

@nzardosh nzardosh enabled auto-merge (squash) September 9, 2025 16:47
@nzardosh nzardosh merged commit e750600 into AliceO2Group:master Sep 12, 2025
17 of 18 checks passed
jmunozme pushed a commit to jmunozme/O2Physics that referenced this pull request Oct 3, 2025
@hanseopark @jmunozme
[PWGJE] Small incorrect code for histogram name and name of table (Al…
56089fb 
…iceO2Group#12925)
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Oct 11, 2025
@hanseopark @jinhyunni
[PWGJE] Small incorrect code for histogram name and name of table (Al…
b1247be 
…iceO2Group#12925)
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
@hanseopark @ThePhDane
[PWGJE] Small incorrect code for histogram name and name of table (Al…
bd9242a 
…iceO2Group#12925)
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
@hanseopark @jloemker
[PWGJE] Small incorrect code for histogram name and name of table (Al…
da34174 
…iceO2Group#12925)
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
@hanseopark
[PWGJE] Small incorrect code for histogram name and name of table (Al…
e9d7ee2 
…iceO2Group#12925)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nzardosh nzardosh nzardosh approved these changes

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@maoyx maoyx Awaiting requested review from maoyx maoyx is a code owner

@fjonasALICE fjonasALICE Awaiting requested review from fjonasALICE fjonasALICE is a code owner

@mfasDa mfasDa Awaiting requested review from mfasDa mfasDa is a code owner

@mhemmer-cern mhemmer-cern Awaiting requested review from mhemmer-cern mhemmer-cern is a code owner

Assignees

No one assigned

Labels

pwgje

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hanseopark @vkucera @nzardosh