-
Notifications
You must be signed in to change notification settings - Fork 17
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
To CMSSW_10_2_4_patch1 #73
Comments
Just happened to ask Phil. Apparently the newer data literally has a different format, so we'll have to run separate releases for the different datasets. |
Actually - central NANOAOD production uses 102X for all Run2 samples (https://twiki.cern.ch/twiki/bin/viewauth/CMS/PdmVAnalysisSummaryTable). It should be possible to process all three years with panda 014. What's the format difference you were referring to? |
There was a change in the way Puppi Weights were packed. It went from something like using a smaller float format to using a short to express a fraction between -1 and 1 (weight = value/short::max) or vice versa. I would have to check. Unfortunately I cleaned up my plot there... |
I see, so we need to add a switch to the filler then. |
I can't seem to duplicate the problem. I don't remember exactly how we fixed it before, but maybe everything is fine now. |
We need to update CMSSW versions for the new data. Of course, CMSSW_10_1_4 -> CMSSW_10_2_4 isn't backwards compatible.
A naive attempt to get things to compile gives:
The text was updated successfully, but these errors were encountered: