coresoftware icon indicating copy to clipboard operation
coresoftware copied to clipboard

Add Module to Build Trigger Clusters

Open ruse-traveler opened this issue 1 year ago • 2 comments

This PR introduces a module a construct Trigger Clusters, various jet trigger patches saved to RawCluster objects for easier downstream analysis. For example, this will enable a user to easily look up the towers which went into a patch that fired a given trigger.

Types of changes

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work for users)
  • [ ] Requiring change in macros repository (Please provide links to the macros pull request in the last section)
  • [x] I am a member of GitHub organization of sPHENIX Collaboration, EIC, or ECCE (contact Chris Pinkenburg to join)

What kind of change does this PR introduce? (Bug fix, feature, ...)

This PR adds a new feature.

TODOs (if applicable)

  • [ ] Finish process_event
    • [ ] Loop over each jet patch
    • [ ] For each patch which fired trigger, construct a RawCluster.
  • [ ] Add piping for saving clusters to node tree
  • [ ] Finish piping for saving clusters to TTree

Links to other PRs in macros and calibration repositories (if applicable)

ruse-traveler avatar May 16 '24 21:05 ruse-traveler

Build & test report

Report for commit aafc65a71af97cf201dd42ed56e25bd0be606cbf: Jenkins on fire


Automatically generated by sPHENIX Jenkins continuous integration sPHENIX             jenkins.io

sphenix-jenkins-ci[bot] avatar May 17 '24 02:05 sphenix-jenkins-ci[bot]

Build & test report

Report for commit b16970bda518b07b4977f8a15f08ec4d99eb5109: Jenkins passed


Automatically generated by sPHENIX Jenkins continuous integration sPHENIX             jenkins.io

sphenix-jenkins-ci[bot] avatar Jun 16 '24 11:06 sphenix-jenkins-ci[bot]

I'm going to go ahead and close this: it's been a while, and I think I might to need to do some refactoring to account for upstream software changes.

ruse-traveler avatar Nov 19 '24 14:11 ruse-traveler