Skip to content

[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC #10947

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
fgrosa merged 3 commits into from
Apr 23, 2025
Merged

[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC #10947

fgrosa merged 3 commits into from
Apr 23, 2025

Conversation

@apalasciano
Copy link
Collaborator

@apalasciano apalasciano commented Apr 23, 2025 •
edited
Loading

@apalasciano apalasciano marked this pull request as draft April 23, 2025 09:50
@apalasciano apalasciano marked this pull request as ready for review April 23, 2025 09:53
@zhangbiao-phy
Copy link
Collaborator

zhangbiao-phy commented Apr 23, 2025

@apalasciano Thanks! Good to me! Let's see if @fgrosa have comments.

@alibuild
Copy link
Collaborator

alibuild commented Apr 23, 2025

Error while checking build/O2Physics/o2 for a2ebb82 at 2025-04-23 12:54:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/10947-slc9_x86-64/0/EventFiltering/PWGHF/HFFilterHelpers.h:1955:28: error: redefinition of default argument for 'bool is4ThetaC'
ninja: build stopped: subcommand failed.

Full log here.

@alibuild
Copy link
Collaborator

alibuild commented Apr 23, 2025

Error while checking build/O2Physics/o2 for abad8eb at 2025-04-23 14:05:

## sw/BUILD/O2Physics-latest/log
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
Error in cling::AutoLoadingVisitor::InsertIntoAutoLoadingState:
/sw/SOURCES/O2Physics/10947-slc9_x86-64/0/EventFiltering/PWGHF/HFFilter.cxx:986:24: error: 'isSelectedProtonFromLcResoOrThetaC' was not declared in this scope
/sw/SOURCES/O2Physics/10947-slc9_x86-64/0/EventFiltering/PWGHF/HFFilter.cxx:1071:22: error: 'isSelectedProtonFromLcResoOrThetaC' was not declared in this scope
ninja: build stopped: subcommand failed.

Full log here.

@fgrosa fgrosa enabled auto-merge (squash) April 23, 2025 13:13
@fgrosa fgrosa merged commit a5f6c95 into AliceO2Group:master Apr 23, 2025
12 of 13 checks passed
bghanley1995 pushed a commit to bghanley1995/O2Physics that referenced this pull request May 5, 2025
@apalasciano @bghanley1995
[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC (Al…
b1d8220 
…iceO2Group#10947)
@apalasciano apalasciano deleted the LcRes branch May 9, 2025 12:49
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request May 11, 2025
@apalasciano @jinhyunni
[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC (Al…
1992d05 
…iceO2Group#10947)
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request May 11, 2025
@apalasciano @jinhyunni
[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC (Al…
cdff100 
…iceO2Group#10947)
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request May 17, 2025
@apalasciano
[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC (Al…
98277d3 
…iceO2Group#10947)
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
@apalasciano @ariedel-cern
[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC (Al…
70d9db9 
…iceO2Group#10947)
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
@apalasciano @smaff92
[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC (Al…
a09cd7c 
…iceO2Group#10947)
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
@apalasciano @CesarOmarRA
[PWGHF,Trigger] Separate proton pt threshold for LcRes and ThetaC (Al…
eb341e6 
…iceO2Group#10947)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fgrosa fgrosa fgrosa approved these changes

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

@zhangbiao-phy zhangbiao-phy Awaiting requested review from zhangbiao-phy zhangbiao-phy is a code owner

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

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

Assignees

No one assigned

Labels

pwghf PWG-HF trigger

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@apalasciano @zhangbiao-phy @alibuild @fgrosa