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
Update for keeping gen-level nuclear info in AOD #12344
Conversation
A new Pull Request was created by @yetkinyilmaz for CMSSW_7_5_X. Update for keeping gen-level nuclear info in AOD It involves the following packages: Configuration/Applications @vciulli, @civanch, @covarell, @danduggan, @boudoul, @franzoni, @mdhildreth, @fabozzi, @cmsbuild, @srimanob, @thuer, @bendavid, @davidlange6, @govoni can you please review it and eventually sign? Thanks. |
Are these changes already in 80x? |
No, in 80X, there will be a different implementation. On 10/11/15 13:41, Josh Bendavid wrote:
|
please test |
The tests are being triggered in jenkins. |
+1 |
+1 |
Update for keeping gen-level nuclear info in AOD
@yetkinyilmaz @davidlange6 @bendavid @fabozzi @civanch @Dr15Jones It is easy to prove that this PR broke step2 of relvals 140.1 and 140.4 in CMSSW_7_5_X. So, what should be done about this? |
Thanks for the warning Bill, and sorry for the trouble. |
This pull-request contains a fix in order to keep all nuclear information (npart, ncoll, reaction plane) when the HepMCProduct is dropped.
This should be considered as a bug-fix in 75X, the production of the "heavyIon" object has to be in the digi step in order to be usable for some large GEN-SIM samples already produced.
The implementation for CMSSW_8_0_X will be different, it will be added to the gen step, in a more elegant way.