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

Missing Seeds in workflows #1136

Closed
aloeliger opened this issue Jun 13, 2023 · 8 comments
Closed

Missing Seeds in workflows #1136

aloeliger opened this issue Jun 13, 2023 · 8 comments
Labels
Issue Understood Log Errors For output and log related issues Phase-1 Pertains to phase-1 development Solution Understood

Comments

@aloeliger
Copy link

What

We have been notified that several seeds seem to be missing in standard relval workflows, specifically L1_SingleJet60er2p5 and L1_SingleJet60_FWD3p0. These are hardcoded as a part of the seeds checked for uGT timing for prescaled seeds here. We should recommend a replacement or removal of these seeds.

Recipe to recreate

Run 2023 relval 141.008

@aloeliger aloeliger added Phase-1 Pertains to phase-1 development Log Errors For output and log related issues labels Jun 13, 2023
@eyigitba
Copy link

eyigitba commented Jun 14, 2023

Hi @aloeliger , I think we should do the following replacements:

Remove:
L1_SingleJet60
L1_SingleJet60er2p5
L1_SingleJet60_FWD3p0

Add:
L1_SingleJet35
L1_SingleJet35er2p5
L1_SingleJet35_FWD2p5

@lathomas @caruta can you confirm this looks good to you?

@aloeliger
Copy link
Author

@lathomas, @cartua I would like to start cleaning up the L1 logs which are notably verbose right now. Unless you have any major objections, I will try to commit Efe's suggestion at some point today.

@lathomas
Copy link

Sounds good for me, thanks.

@caruta
Copy link

caruta commented Jun 15, 2023

Hi, I'm fine with it too. Thanks!

@aloeliger
Copy link
Author

@eyigitba @lathomas @caruta, Apparently L1_ETT1200 is also missing. Do we have a suitable replacement for that?

@caruta
Copy link

caruta commented Jun 16, 2023

Hi @aloeliger, we're going to remove L1_ETT1200 starting from the next L1 menu: 2023_v1_2_0, which will be deployed before the end of this month. FYI, here you can find the complete list of seeds that will be removed in the next menu.
I think we can replace L1_ETT1200 with L1_ETT1600: @lathomas, @eyigitba do you agree?

@eyigitba
Copy link

Yes, I think that should be fine

@aloeliger
Copy link
Author

Closed in cms-sw#41972

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue Understood Log Errors For output and log related issues Phase-1 Pertains to phase-1 development Solution Understood
Projects
None yet
Development

No branches or pull requests

4 participants