Skip to content
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

[DQM] Prompt matrix redefinition #4617

Closed
wants to merge 2 commits into from
Closed

Conversation

jfernan2
Copy link

@jfernan2 jfernan2 commented Oct 11, 2021

Agreed on PPD coord 08.09.2021.

Reduced:

  • L1TMon to SingleMuon, SingleElectron/EG and ZB
  • TAU to SingleMuon, SingleElectron/EG and TAU
  • TRK to SingleMuon, SingleElectron/EG, DoubleMuon, JetHT, JetMET and ZB, by creating commonReduced which excludes TRK
  • CTPPS to DoubleMuon, SingleElectron/EG y ZB

https://docs.google.com/presentation/d/1AF65xzq7T70Yt-0o_OV47YTFBv-kbvHZDlRmUSCZsAw/edit#slide=id.geedc047305_0_0

This PR goes in sync with: cms-sw/cmssw#35605

Reduced:
- L1TMon to SingleMuon, SingleElectron/EG and ZB
- TAU to SingleMuon, SingleElectron/EG and TAU
- TRK to SingleMuon, SingleElectron/EG, DoubleMuon, JetHT, JetMET and ZB, by creating commonReduced which excludes TRK
- CTPPS to DoubleMuon, SingleElectron/EG y ZB
@cmsdmwmbot
Copy link

Can one of the admins verify this patch?

@germanfgv
Copy link
Contributor

Hi @jfernan2. You are modifying our production configuration file. If you want us to test a configuration, you should modify the ReplayOfflineConfiguration.py file. Also, please follow our PR template and provide the information that is requested there.
PR template

I'm closing this PR for now, but feel free to reopen once you have the proper modifications and provide context to the changes.

@germanfgv germanfgv closed this Oct 11, 2021
@francescobrivio
Copy link
Contributor

@jfernan2
Just for my understanding: this is planned for CMSSW_12_1_X right?

@jfernan2
Copy link
Author

@jfernan2 Just for my understanding: this is planned for CMSSW_12_1_X right?

Yes, correct!

@jfernan2
Copy link
Author

Hi @jfernan2. You are modifying our production configuration file. If you want us to test a configuration, you should modify the ReplayOfflineConfiguration.py file. Also, please follow our PR template and provide the information that is requested there. PR template

I'm closing this PR for now, but feel free to reopen once you have the proper modifications and provide context to the changes.

Thanks @germanfgv
I will do so

@francescobrivio
Copy link
Contributor

@jfernan2 Just for my understanding: this is planned for CMSSW_12_1_X right?

Yes, correct!

Ok thanks!
So tesing with a replay should be fine (provided Tier0 experts agree and have time/resources at the moment),
but we should not deploy this in production at the moment (sorry to be nagging, I'm just speaking as this weeks' ORM 😃 ).

@jfernan2
Copy link
Author

OK, @francescobrivio then I believe I had a wrong understanding of this
On the other hand, @germanfgv it looks like ReplayOfflineConfiguration.py is not in sync with OfflineConfiguration.py so, it is not clear to me how to proceed.

This PR is reduction of resources for Tier0, we are dropping sequences not needed by subsystems in several PDs

@jfernan2
Copy link
Author

Anyway, I have created this PR on the Replay file, reflecting the changes from this one:
#4619
I am not sure about the GTs though @francescobrivio
Thank you

@francescobrivio
Copy link
Contributor

OK, @francescobrivio then I believe I had a wrong understanding of this On the other hand, @germanfgv it looks like ReplayOfflineConfiguration.py is not in sync with OfflineConfiguration.py so, it is not clear to me how to proceed.

I think this were the correct changes if you wanted to deploy the new DQM matrix in production. But since we are currently using 12_0_X, the prod configuration should not be update now.

This PR is reduction of resources for Tier0, we are dropping sequences not needed by subsystems in several PDs

Yes I believe this is more than welcome in Tier0 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants