[L1T] Update recommended geometry tag in Phase2 L1TNanoAOD producer README to D121 (Phase2)#49870
[L1T] Update recommended geometry tag in Phase2 L1TNanoAOD producer README to D121 (Phase2)#49870RobertJWard wants to merge 2 commits intocms-sw:masterfrom
Conversation
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49870/47570 |
|
A new Pull Request was created by @RobertJWard for master. It involves the following packages:
@BenjaminRS, @battibass, @cmsbuild, @ftorrresd, @quinnanm can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
please test |
|
+1 Size: This PR adds an extra 20KB to repository Comparison SummarySummary:
|
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49870/47575
|
|
Pull request #49870 was updated. @BenjaminRS, @battibass, @cmsbuild, @ftorrresd, @quinnanm can you please check and sign again. |
|
@RobertJWard, can you please collapse your commits into a single one (as described here) before we approve the PR? |
ede9297 to
ff05c25
Compare
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49870/47626 |
|
Pull request #49870 was updated. @BenjaminRS, @battibass, @cmsbuild, @ftorrresd, @quinnanm can you please check and sign again. |
|
Hi @battibass I've now squashed the commits (though an extra merge commit was added as the conditions update T33 -> T35 was also made in parallel in #49790, resulting in a conflict) |
|
please test |
|
+1 |
|
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
|
Milestone for this pull request has been moved to CMSSW_17_0_X. Please open a backport if it should also go in to CMSSW_16_1_X. |
PR description:
Updates the recipe in the README for the Phase 2 L1TNanoAOD producer tool to the latest phase 2 recommended geometry of D121 (previously D110).
PR validation:
None required - simple text change to a markdown file, with no impact on the code itself.