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
Run3-gem66 Update the production cut files for run3 muon system #36551
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36551/27484
|
A new Pull Request was created by @bsunanda (Sunanda Banerjee) for master. It involves the following packages:
@civanch, @Dr15Jones, @makortel, @cvuosalo, @ianna, @mdhildreth, @cmsbuild can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild Please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-656402/21412/summary.html Comparison SummarySummary:
|
@bsunanda Please add to the PR description that the file versions in this PR have not yet been uploaded to the DB, so it is safe to modify them without creating new versions (if that is true). |
@cvuosalo Added in the description. |
+1 |
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
@bsunanda , please, clarify situation with cuts for protons. In this PR you removing cuts for proton from sub-regions. I am not sure if this is optimal. |
@cmsbuild Please test |
@civanch ProdCutsForProtons were not defined in many of the earlier production cut file for muon. Now I have added 10 mm for Muon region and 1mm for all the subregions |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36551/27524
|
Pull request #36551 was updated. @civanch, @Dr15Jones, @makortel, @cvuosalo, @ianna, @mdhildreth can you please check and sign again. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-656402/21431/summary.html Comparison SummarySummary:
|
+1 |
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
Update the production cut files for run3 muon system. The file versions in this PR have not yet been uploaded to the DB. So it is safe to modify the files without creating new versions.
PR validation:
Use the runTheMatrix test workflows
if this PR is a backport please specify the original PR and why you need to backport that PR:
Nothing special