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
Backport to 13_0_X of #41275 Introduce (Robust)ParTAK4 jet tagger, DeepJet model update for Run 3, remove DeepCSV from nano #41434
Conversation
A new Pull Request was created by @AnnikaStein (Annika Stein) for CMSSW_13_0_X. It involves the following packages:
@swertz, @vlimant, @clacaputo, @cmsbuild, @simonepigazzini, @mandrenguyen can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Pull request #41434 was updated. @swertz, @vlimant, @clacaputo, @cmsbuild, @simonepigazzini, @mandrenguyen can you please check and sign again. |
Pull request #41434 was updated. @swertz, @vlimant, @clacaputo, @cmsbuild, @simonepigazzini, @mandrenguyen can you please check and sign again. |
backport of #41275 |
enable nano |
please test |
-1 Failed Tests: RelVals RelVals-INPUT RelVals-NANO AddOn RelVals
Expand to see more relval errors ...RelVals-INPUT
Expand to see more relval errors ...
RelVals-NANO
Expand to see more relval errors ...AddOn Tests
Expand to see more addon errors ... |
Pull request #41434 was updated. @swertz, @vlimant, @clacaputo, @cmsbuild, @simonepigazzini, @mandrenguyen can you please check and sign again. |
We would need to test this with the model files (cms-data/RecoBTag-Combined#51 - updated to use the adversarial model weights for ParTAK4, otherwise same usage with DeepJet for Run 3). The suggested changes from the original PR (c857a54) have been included here as well today for the backport. |
Pull request #41434 was updated. @swertz, @vlimant, @clacaputo, @cmsbuild, @simonepigazzini, @mandrenguyen can you please check and sign again. |
please test |
Noticed there were additional keywords which are unknown to the modifier, which resulted in these workflows failing (https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-58a9be/32341/runTheMatrixNANO-results/wf_errors.txt)
apologies for that, must have happened while adapting to the recently merged PR |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-58a9be/32346/summary.html 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: Comparison SummarySummary:
NANO Comparison SummarySummary:
Nano size comparison Summary:
|
Are there any open points for this backport? From our perspective, the tests produced the expected results, similar to the ones in the PR for master, where the outdated DeepCSV gets removed successfully, replaced by a new version of DeepJet (which causes differences in the comparisons) and RobustParTAK4 (which is not yet rerun for early-Run3 samples as we anticipate that re-mini happens). |
Hi @AnnikaStein |
By the way the model's PR has not yet been merged : cms-data/RecoBTag-Combined#51 |
I just prepared the backport cmsdist PR for 13_0_X: cms-sw/cmsdist#8493 |
test parameters: |
+1 |
+reconstruction |
+1 |
PR description:
See description of #41275 - new tagger for Run 3 for MiniAOD and NanoAOD, update of DeepJet for Run 3, and removing outdated DeepCSV for Run 3.
Additions are verbatim copies of original PR, modifications have been updated to be in line with latest PNet additions into the same packages.
PR validation:
Compilation, basic checks similar to original PR.
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:
Backport of #41275