-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
[NanoAOD] Add ParticleNet-based Muon ID #46153
Conversation
cms-bot internal usage |
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46153/41945 Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46153/41946 Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46153/41947 Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46153/41948 |
Pull request #46153 was updated. @cmsbuild, @ftorrresd, @hqucms, @jfernan2, @mandrenguyen can you please check and sign again. |
please test |
-1 Failed Tests: UnitTests RelVals RelVals-INPUT RelVals-NANO The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: You can see more details here: Unit TestsI found 1 errors in the following unit tests: ---> test test-das-selected-lumis had ERRORS RelVals
Expand to see more relval errors ...RelVals-INPUT
Expand to see more relval errors ...RelVals-NANO |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46153/42012 |
Pull request #46153 was updated. @cmsbuild, @ftorrresd, @hqucms, @jfernan2, @mandrenguyen can you please check and sign again. |
please test |
-1 Failed Tests: RelVals RelVals-INPUT RelVals-NANO The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: RelVals
Expand to see more relval errors ...RelVals-INPUT
Expand to see more relval errors ...RelVals-NANO |
please close Superseded by #46273. |
[NanoAOD] Add ParticleNet-based Muon ID (Rework of #46153)
PR description:
This PR adds ParticleNet-based Muon ID for NanoAODv15.
The feature was discussed during Muon POG meeting (23/09/2024) and it will be discussed during XPOG meeting (2/10/2024)
PR validation:
I've performed a local running test to see that the mva is loaded and nanoAOD is in fact produced.