-
Notifications
You must be signed in to change notification settings - Fork 13
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 METsignificance #268
add METsignificance #268
Conversation
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.
So IIUC the whole MET corrections are now done in the JetMET tools ? These are already in the CMSSW release we are using, no need to check out anything ?
please test |
As expected the check fails for the
|
They are not in the release, but they are in the branch we're already merging for the MET recipe (the tricky part is if @alesaggio and I got the call to the JetMET tools right... but since the MET branch didn't change, we assumed it worked ;-) ). We could also have pulled the relevant parts out of the JetMET tools... in principle there should only be one or a few modules to add, but we tried the simple option first :-) |
Yes @OlivierBondu I'm doing it |
please test |
tracked at #257 |
No description provided.