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
Switch by default to new flavour tool based on Hadrons. Update of exampl... #8056
Conversation
A new Pull Request was created by @acaudron for CMSSW_7_4_X. Switch by default to new flavour tool based on Hadrons. Update of exampl... It involves the following packages: DQMOffline/RecoB @cmsbuild, @danduggan, @nclopezo, @deguio can you please review it and eventually sign? Thanks. |
This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @Degano, @ktf, @smuzaffar |
@davidlange6, can you please merge this PR (it's been fully approved for more than a week and its 75X equivalent #8049 has already been merged) |
What's the hold up here? |
This PR got left out of the final 740 release. Was there a specific reason for that and can this PR be included in a later 74X release? |
@ferencek - pushing new features back to 74x needs some validation first in 75x (so pre1) - we can back port these changes to 74x in the coming weeks. |
@davidlange6, thank you for clarifying. |
Based on the previous discussion, I assume this PR can now be merged. |
+1 |
Switch by default to new flavour tool based on Hadrons. Update of exampl...
...e scripts.
Same as PR#8049 for 75x:
Switch for the Btag DQM to the new jet flavour tool described here:
https://twiki.cern.ch/twiki/bin/view/CMSPublic/SWGuideBTagMCTools#Hadron_based_jet_flavour_definit
and already default in PAT.
Examples scripts are also updated accordingly.
Possibility to use the old flavour tool is conserved, especially as the new tool, targeting PFJets, cannot be applied on some jet collections as CaloJets.