Update NGJet model version#50975
Conversation
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50975/49381 |
|
A new Pull Request was created by @Chriisbrown for master. It involves the following packages:
@BenjaminRS, @cmsbuild, @quinnanm can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
test parameters:
|
|
please test |
|
-1 Failed Tests: UnitTests RelVals 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: Failed Unit TestsI found 1 errors in the following unit tests: ---> test testProduceNanoHLT had ERRORS Failed RelVals |
|
please test |
I guess cms-data/RecoHGCal-TICL#12 didn't make the last build. |
|
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
|
+l1 |
|
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, @ftenchini, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
PR description:
This PR updates the default model version for the L1TSC4NGJetModel from v0 to v1_0_1
Corresponding cms-hls4ml emulator: https://github.com/cms-hls4ml/L1TSC4NGJetModel
Corresponding PR to cms-dist: cms-sw/cmsdist#10485
This PR is for CMSSW 17 as this is the version after which the above cms-dist PR was merged into
Changes to the model inputs were made in this PR: #50174 which necessitated an update to the model