-
Notifications
You must be signed in to change notification settings - Fork 62
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
Tuning up TpcRS for AuAu19GeV 2019 #184
Conversation
Dmitry and Jason, it seems the changes involves some updates in StDb/idl and g2t tables. Could you please take a review and comment/approve this PR? Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approve this pull request, but take note of an issue it raises with the event data model.
Changes to pams/sim/idl/g2t_tpc_hit.idl introduce data members which are used to store intermediate calculations from TpcRS. This adds an additional responsibility to the hit structure. It represents both the MC event data model AND tpc internal calculations.
In an ideal design, these responsibilities would be kept separate. A parallel data structure in StTpcRSMaker would be preferred over changing the simulation data format.
(cherry picked from commit b91e0fd)
No description provided.