-
Notifications
You must be signed in to change notification settings - Fork 68
[HI Preparation] Replay for new patched release, 15_1_0_patch3 #5170
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
Conversation
|
Can one of the admins verify this patch? |
etc/HIProdOfflineConfiguration.py
Outdated
| 'default': "CMSSW_14_1_5_patch1", | ||
| 'acqEra': {'Run2024J': "CMSSW_14_1_4_patch1"}, | ||
| 'default': "CMSSW_15_1_0_patch3", | ||
| 'acqEra': {'Run2025A': "CMSSW_15_1_0_patch3"}, |
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 it be HIRun2025A instead for acquisition era?
Do we need this line? I see it was removed in last replay PR.
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.
I mean in the following PR,
https://github.com/dmwm/T0/pull/5166/files
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.
Hi @srimanob, thanks for the comment. I have changed and commented. I'm rather unsure if the replay uses the prod file. That being said, indeed, past PR had removed that. The original file is from last year's prod setup, I suppose. Hence, I assume that the acqEra was run with no HI in the name. However, this would be better if T0 clarifies what that parameter actually does.
Best, Jordan
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.
@srimanob @jordan-martins thanks for the discussion here.
This parameter allows us to select the cmssw version by acquisition era or run number. If we have to switch version with an era change, then I can simply remove the comment here, put the previous era as key with correseponding cmssw version as value. All runs from the new era will release reco with the default cmssw version.
|
I see that @LinaresToine just merge the previous HI replay PR to master, so this PR needs to rebase, to resolve the conflicts. |
|
Thanks @srimanob, I am working on the conflicts now |
|
Actually, I think I can't resolve the conflicts without merging. @jordan-martins could you please rebase and update the PR? |
Replay Request
Requestor
ORM Shifter
Describe the configuration
Purpose of the test
This patched release includes some PRs that address Skims, reco, and alignment updates.
T0 Operations cmsTalk thread
Tier0 Operations cmsTalk Forum