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

To CMSSW_10_2_4_patch1 #73

Open
dabercro opened this issue Oct 23, 2018 · 5 comments
Open

To CMSSW_10_2_4_patch1 #73

dabercro opened this issue Oct 23, 2018 · 5 comments

Comments

@dabercro
Copy link
Collaborator

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:

@dabercro
Copy link
Collaborator Author

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.

@yiiyama
Copy link
Collaborator

yiiyama commented May 29, 2019

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?

@yiiyama yiiyama reopened this May 29, 2019
@dabercro
Copy link
Collaborator Author

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...

@yiiyama
Copy link
Collaborator

yiiyama commented May 29, 2019

I see, so we need to add a switch to the filler then.

@dabercro
Copy link
Collaborator Author

I can't seem to duplicate the problem. I don't remember exactly how we fixed it before, but maybe everything is fine now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants