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
Run2-gex128 Rearrange 2017 scenarios in view of updates foreseen #37593
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37593/29353
|
A new Pull Request was created by @bsunanda (Sunanda Banerjee) for master. It involves the following packages:
@civanch, @Dr15Jones, @makortel, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @AdrianoDee, @srimanob can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild Please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37593/29354
|
Pull request #37593 was updated. @civanch, @Dr15Jones, @makortel, @cvuosalo, @ianna, @mdhildreth, @cmsbuild, @AdrianoDee, @srimanob can you please check and sign again. |
@cmsbuild Please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-0b75dd/23966/summary.html Comparison SummarySummary:
|
You're changing Extended2017 to Extended2017Plan0. Is "Extended2017" not currently used in any code? |
I plan to use the new 2017 scenario there. I consulted with HCAL DPG SW expert about this. The current status is confusing. The defaukt usage is with 2017Plan1. The new 2017 scenario also uses this but with something which can work with dd4hep. As soon as this IB is integrated I shall migrate the versions in HcalCommonData to CMSCommonData. This will be for 2017 as well as 2018. |
All current 2017 applications use the DB version. |
+1 |
@srimanob Please sign this PR |
Do we need this urgently? My proposal is to discuss first in the SIM meeting? |
@bsunanda , all, would it be more safe not to rename whatever file but create a new one with clear name like "DD4hep" inside? The alternative may be to edit existing file, this from one hand may be risky but reduce necessity to edit downstream files - we may come to situation of expansion of modifications. A safe variant may be to preserve old files with different name and edit existing. |
@srimanob Please sign it now |
In a subsequent PR I shall introduce what we discussed in the SIM meeting. |
+Upgrade There will be a follow up PR as discussed in the SIM meeting, https://indico.cern.ch/event/1153217/, on migration Run-2 to DD4hep. |
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 |
@bsunanda After this PR gets merged, there are some unit test failures appearing in the IB test, please have a look and fix: ----- Begin Fatal Exception 25-Apr-2022 15:12:57 CEST----------------------- |
PR description:
Rearrange 2017 scenarios in view of updates foreseen
PR validation:
Use the runTheMatrix test workflows as well as cfg'sin test area
if this PR is a backport please specify the original PR and why you need to backport that PR:
Nothing special