forked from cms-sw/cmssw
-
Notifications
You must be signed in to change notification settings - Fork 10
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
New MVA-based Tau-Ids (update) #146
Merged
arizzi
merged 14 commits into
cms-nanoAOD:master
from
mbluj:CMSSW_9_4_X_MVAIso2017v2ForNano
Apr 4, 2018
Merged
Changes from 5 commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
d401c19
Merged master from repository cms-nanoAOD with cms-merge-topic
mbluj 33919f8
Add payloads for tau-Id MVAIso 2017v2
mbluj 7bc2e4a
Add MVAIso tau-Id 2017v2
mbluj 505a673
Move definitions of new MVA tau-Ids to a specific cff file
mbluj 7500553
Use correct eras, add MVAIso2017v1
mbluj 761163c
Add cut-based isolation WPs
mbluj c87de85
correct eras, comment out idCombIsodR03 due to incorrect behavior
mbluj 4b6743d
further era corrections
mbluj 6c4b146
fix bug in eras handling
mbluj 46ec6cf
add rawIso dR=0.3, remove cut-base WPs and footprintCorr
mbluj 0312498
remove MVAIso oldDM dR=0.3 and newDM with 2015 training
mbluj b5e007e
remove instead commenting out
mbluj 5df8704
keep all MVAIso 2015 Tau-Ids and remove 2017v1/2 ones for 80X
mbluj 60b48d2
Add era modifiers to NanoDQM
mbluj File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
don't use
run2_miniAOD_94XFall17
eraHave the default config to be that for 94XMiniAODv2, and use
run2_nanoAOD_94XMiniAODv1
to adapt to MiniAOD v1(I'm putting just one comment but it is true for all uses of this era in the file)
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.
OK, thanks for letting me know, I will correct it.