-
Notifications
You must be signed in to change notification settings - Fork 4.6k
NanoAODv15-Lite setup for Run2UL and 2022/2023 #48104
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
Conversation
|
cms-bot internal usage |
|
enable nano |
|
please test |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-48104/44854 |
|
+1 Size: This PR adds an extra 36KB to repository Comparison SummarySummary:
NANO Comparison SummarySummary:
Nano size comparison Summary:
|
|
@cms-sw/btv-pog-l2 @cms-sw/jetmet-pog-l2 @cms-sw/tau-pog-l2 @cms-sw/muon-pog-l2 @cms-sw/egamma-pog-l2 |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-48104/44873 |
|
Pull request #48104 was updated. |
|
please test |
|
+1 Size: This PR adds an extra 20KB to repository Comparison SummarySummary:
NANO Comparison SummarySummary:
Nano size comparison Summary:
|
|
+1 |
|
@hqucms I am not sure about this TODO item: Need to check the JERC tags for the GTs. Is it apart from this PR? |
|
are there test workflows included in the nano matrix for this, or else? |
Yes the existing Run2/106X ( |
And the full set of cmsDriver commands are posted in #48176 (comment). |
And do we understand why the difference in rate is now performing worse for those wfs? Thanks |
@jfernan2 It's because we need to re-run Puppi and recluster jets, so the timing increases quite a bit. |
|
+1 |
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @mandrenguyen, @sextonkennedy, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
[15_0_X, backport of #48104] NanoAODv15-Lite setup for Run2UL and 2022/2023
|
+1 |
PR description:
This PR sets up a
NanoAODv15-Liteconfiguration to produce v15-like NANO directly from Run2UL MiniAODv2 and Run3 2022/2023 130X MiniAOD, by:Muon_softMvaRun3on-the-flyFrom some quick comparison to a reMINI+reNANO workflow, overall reasonable agreement was observed, however small differences exist here and there (some are expected due to the precision differences between reco::PFCandidate and pat::packedCandidate).
TODOs:
LowPtElectroncollection -- comments are welcome.Muon_softMvaRun3is not filled -- Fixed by re-running the MVA on-the-fly.PR validation:
Standard NANO workflows.
If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:
To be backported to 15_0_X.