-
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
Restore CSC DQM monitor outside Muon POG sequence #46353
Comments
cms-bot internal usage |
A new Issue was created by @fabiocos. @Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
@ptcox FYI |
1 similar comment
@ptcox FYI |
assign dqm |
New categories assigned: dqm @antoniovagnerini,@nothingface0,@rvenditti,@syuvivida,@tjavaid you have been requested to review this Pull request/Issue and eventually sign? Thanks |
As an interim solution, pending a better organization of the muon DQM sequences to separate |
Hi everyone. I tried to look into this unwanted coupling with Muon POG DQM and would like to suggest the following:
|
Hi Tim, thanks a look for taking the time to look into this. It looks good to me; I will implement your suggestion, test it, and let you know |
Thanks for looking it. I'm now concerned I don't understand how things really work though - I am getting confused between |
Yes, I am wrong: one needs a |
Something that seems to work is simply adding the
and
Though I didn't touch the harvesting part at all, so I have to fully convince myself that this is enough |
Yes, thanks for trying. I think it may be that |
that's correct. Normally any |
Thanks for that @mmusich. |
IN CMSSW_14_0_18 the PR #46298, backport of #46293, has been integrated with the purpose of alleviating the memory issues faced by Tier0 in express harvesting workflows. It turns out that his updated has removed also CSC DQM monitoring, as reported in https://cms-talk.web.cern.ch/t/missing-folder-in-streamexpress-run2024i-express-v2-dqmio/52718/5 , since this sequenced has been not appropriately embedded in the Muon POG sequence, see https://github.com/cms-sw/cmssw/pull/46298/files#r1796815700 .
@rseidita is aware of the issue, and agreed to relocate the involuntarily removed sequence so as to restore the functionality. This issue is meant to record the problem and track its solution. As soon as the fix is available and backported to the CMSSW_14_0_X branch, a patch will be needed to restore the missing functionality in the Tier0.
The text was updated successfully, but these errors were encountered: