-
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
[14_0_X] Introduce Unified Particle Transformer AK4 jet tagger (Backport) #44660
[14_0_X] Introduce Unified Particle Transformer AK4 jet tagger (Backport) #44660
Conversation
A new Pull Request was created by @AlexDeMoor for CMSSW_14_0_X. It involves the following packages:
@mandrenguyen, @jfernan2, @vlimant, @cmsbuild, @hqucms can you please review it and eventually sign? Thanks. cms-bot commands are listed here
|
cms-bot internal usage |
enable nano |
test parameters:
|
backport of #44641 |
please test |
This dataset is not stored in T2_CH_CERN anymore so the dasgoclient will return a blank list. Local test works well after removing this site requirement and adding the backport mentioned in 1. |
please abort |
please test with #44627 |
-1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ab0c66/38715/summary.html This pull request cannot be automatically merged, could you please rebase it? |
9d40b1f
to
61cf755
Compare
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ab0c66/38863/summary.html Comparison SummarySummary:
NANO Comparison SummarySummary:
Nano size comparison Summary:
|
REMINDER @rappoccio, @antoniovilela, @sextonkennedy: This PR was tested with #44723, please check if they should be merged together |
@cms-sw/reconstruction-l2 Could you please review and sign? Thanks! |
type btv, jetmet, tau |
+reconstruction |
This pull request is fully signed and it will be integrated in one of the next CMSSW_14_0_X IBs (tests are also fine) and once validation in the development release cycle CMSSW_14_1_X is complete. This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @rappoccio, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
@cms-sw/orp-l2 I opened the cms-dist PR cms-sw/cmsdist#9148, which should be merged together. |
+1 |
…4-04-07-2300 [14_0_X] Introduce Unified Particle Transformer AK4 jet tagger (Backport)
PR description:
This PR introduce UnifiedParticleTransformerAK4 a novel inclusive tagger for jet. This network will perform an inclusive tagging combining b/c/tau/lep tagging with jet energy regression (both the regression and the resolution estimation via quantile regression). The model is trained with a specific robust training combining improved adversarial training and domain adaptation for reducing the impact of the data/MC disagreement on the final performance. The output nodes of the domains are also kept for exploring the possibility of efficiency mapping and their impact later.
An overview of the method can be seen at: https://indico.cern.ch/event/1368069/contributions/5793148/
A focus on the novel adversarial training is described here: https://indico.cern.ch/event/1372038/#3-adversarial-training-for-par
The preliminary results of the model were shown in the following meeting: https://indico.cern.ch/event/1397392/#17-preliminary-results-of-part
The final results will be shared this Monday: https://indico.cern.ch/event/1403350/#3-part-2024-final-results-and
This PR requires the associated ONNX model which has been submitted in the adequate RecoBTag-Combined repo: cms-data/RecoBTag-Combined#57
For your information: a last training is ongoing trying to improve the current performance via an enriched dataset. A modification of the final model could occur. This will only affect the RecoBTag-Combined PR, not this one.
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:
Yes this is the backport of #44641