-
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
Changing Commissioning 2020 to Commissioning 2021 with new run # and dataset. #34643
Conversation
Creating a new step for RunCosmics2021 with run number 343498 and dataset /ExpressCosmics/Commissioning2021-Express-v1/FEVT Follow up of the PR cms-sw#34613.
removing RunCosmics2020
changing RunCosmics2020 to RunCosmics2021 in standard wf 138.1 138.2
Updating comment
please test Hello @boudoul , do we need this PR urgently? 12_0_0 is closing now. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34643/24246
|
A new Pull Request was created by @bbilin (Bugra Bilin) for master. It involves the following packages:
@jordan-martins, @chayanit, @bbilin, @wajidalikhan, @kpedro88, @srimanob, @kskovpen can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-69870c/17252/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:
|
@@ -490,8 +490,9 @@ | |||
steps['RunCosmics2015C']={'INPUT':InputInfo(dataSet='/Cosmics/Run2015C-v1/RAW',label='2015C',run=[256259],events=100000,location='STD')} | |||
steps['RunCosmics2016B']={'INPUT':InputInfo(dataSet='/Cosmics/Run2016B-v1/RAW',label='2016B',run=[272133],events=100000,location='STD')} | |||
|
|||
### LS2 - MWGR#5 2020 - CSC, DAQ, DCS, DQM, DT, ECAL, GEM, HCAL, RPC, TCDS, TRACKER, TRG ### | |||
steps['RunCosmics2020']={'INPUT':InputInfo(dataSet='/ExpressCosmics/Commissioning2020-Express-v1/FEVT',label='2020GR5',ls={338714: [[1, 10000]]},events=100000,location='STD')} | |||
### LS2 - MWGR 2021 - CSC, DAQ, DCS, DQM, DT, ECAL, GEM, HCAL, RPC, TCDS, TRACKER, TRG ### |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should you update this commen also?
With 343498, we have
CSC, DAQ, DCS, DQM, DT, ECAL, GEM, HCAL, L1SCOUT, PIXEL, RPC, TCDS, TRACKER, TRG
Information can be found in
https://cmsoms.cern.ch/cms/runs/report?cms_run=343498&cms_run_sequence=GLOBAL-RUN
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done. thanks for correcting
correcting subsystems in the run
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34643/24326
|
Please test |
Please abort |
test parameters:
|
Please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-69870c/17393/summary.html Comparison Summary@slava77 comparisons for the following workflows were not done due to missing matrix map:
Summary:
|
+Upgrade This PR is to update the input of the commissioning workflows 138.1 and 138.2 from 2020 to 2021 data. New workflows run successfully. |
+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. @silviodonato, @dpiparo, @qliphy, @perrotta (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
Addition to PR #34613 discussions