Skip to content
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.

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

Add idTruth and qaTruth to StEvent and MuDst FttPoint, FttCluster, and FttRawHit #699

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

jdbrice
Copy link
Contributor

@jdbrice jdbrice commented Jul 30, 2024

Adds idTruth member to StMuFttPoint (used by StFttFastSimMaker) and StMuFttCluster (to be used by "slow" simulator)
mirrors updates to StEvent in #698

@jdbrice
Copy link
Contributor Author

jdbrice commented Jul 30, 2024

I am still not sure the best way to make a PR that depends on another PR in progress... This one fails because the corresponding members in StEvent won't be in until #698 is merged

@jdbrice jdbrice changed the title Add idTruth to MuDst FttPoint and FttCluster Add idTruth and qaTruth to StEvent and MuDst FttPoint, FttCluster, and FttRawHit Aug 16, 2024
@jdbrice
Copy link
Contributor Author

jdbrice commented Aug 16, 2024

commit 93b1fb9 does a few things:

  • adds missing qaTruth everywhere
  • adds idTruth and qaTruth to raw hit classes (needed for the full simulator)
  • bumps version numbers where needed
  • adds id/qaTruth to the summary print outs

Copy link
Contributor

@klendathu2k klendathu2k left a comment

Choose a reason for hiding this comment

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

Adds to the data model... and I see the classdef macros are properly implemented. No issues. LGTM.

@jdbrice jdbrice added the FWD Forward Upgrade label Aug 22, 2024
@akioogawa akioogawa merged commit b06b780 into star-bnl:main Aug 22, 2024
148 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FWD Forward Upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants