Skip to content

Conversation

@nstrangm
Copy link
Contributor

@nstrangm nstrangm commented Mar 9, 2025

  • Add heavy neutral meson (omega, eta') trigger task
  • Add cluster QA histograms to EM cluster skimmer task
  • Add heavy neutral meson utilities header to EM

This trigger task is meant for testing to find the omega peak on the grid. The trigger selectivity will be drastically increased in a future PR by including a femto requirement.

@nstrangm nstrangm changed the title [PWGEM/PhotonMeson, EventFiltering] Add Heavy Neutral Meson Software Trigger Task [PWGEM/PhotonMeson, Trigger] Add Heavy Neutral Meson Software Trigger Task Mar 9, 2025
@github-actions github-actions bot changed the title [PWGEM/PhotonMeson, Trigger] Add Heavy Neutral Meson Software Trigger Task [PWGEM/PhotonMeson,Trigger] Add Heavy Neutral Meson Software Trigger Task Mar 9, 2025
nstrangm and others added 2 commits March 9, 2025 20:32
@alibuild
Copy link
Collaborator

alibuild commented Mar 9, 2025

Error while checking build/O2Physics/o2 for fc59723 at 2025-03-09 21:41:

## 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/10421-slc9_x86-64/0/PWGEM/PhotonMeson/Utils/HNMUtilities.h:165:51: error: increment of read-only variable 'iGG'
/sw/SOURCES/O2Physics/10421-slc9_x86-64/0/PWGEM/PhotonMeson/Utils/HNMUtilities.h:165:51: error: increment of read-only variable 'iGG'
ninja: build stopped: subcommand failed.

Full log here.


namespace ReconstructionType
{
enum ReconstructionType {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you use the "enum PairType" defined in PairUtilities.h?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, thanks! Sorry, I forgot this already existed in the EM code. I changed it to use the PairType from the general utils now.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Danke Schoen!

@dsekihat dsekihat self-requested a review March 10, 2025 10:24
@mpuccio mpuccio merged commit 6b94db7 into AliceO2Group:master Mar 10, 2025
14 of 16 checks passed
MykhailoShamrai pushed a commit to MykhailoShamrai/O2Physics that referenced this pull request Mar 12, 2025
…Task (AliceO2Group#10421)

Co-authored-by: Nicolas Strangmann <nicolas.strangmann@.cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
louisemillot pushed a commit to louisemillot/O2Physics that referenced this pull request Apr 8, 2025
…Task (AliceO2Group#10421)

Co-authored-by: Nicolas Strangmann <nicolas.strangmann@.cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
njacazio pushed a commit that referenced this pull request Apr 11, 2025
…Task (#10421)

Co-authored-by: Nicolas Strangmann <nicolas.strangmann@.cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
EmilGorm pushed a commit to EmilGorm/O2Physics that referenced this pull request Apr 15, 2025
…Task (AliceO2Group#10421)

Co-authored-by: Nicolas Strangmann <nicolas.strangmann@.cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
…Task (AliceO2Group#10421)

Co-authored-by: Nicolas Strangmann <nicolas.strangmann@.cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
alibuild added a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
…Task (AliceO2Group#10421)

Co-authored-by: Nicolas Strangmann <nicolas.strangmann@.cern.ch>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants