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

expand coverage for creation of simGtExtUnprefireable in miniAOD #32304

Open
slava77 opened this issue Nov 26, 2020 · 10 comments
Open

expand coverage for creation of simGtExtUnprefireable in miniAOD #32304

slava77 opened this issue Nov 26, 2020 · 10 comments

Comments

@slava77
Copy link
Contributor

slava77 commented Nov 26, 2020

the initial version integrated in #31985 is making the simGtExtUnprefireable product available only in run2_miniAOD_UL setup.

todo: to figure out the full set of input workflows where tcdsDigis:tcdsRecord is available as input for the miniAOD step and apply that combined era modifier instead of the current run2_miniAOD_UL

@rekovic

@slava77
Copy link
Contributor Author

slava77 commented Nov 26, 2020

assign reconstruction,l1

@cmsbuild
Copy link
Contributor

New categories assigned: reconstruction,l1

@slava77,@rekovic,@perrotta,@jmduarte,@jpata you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

A new Issue was created by @slava77 Slava Krutelyov.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@cecilecaillol
Copy link
Contributor

Fixed in #31985

@jpata
Copy link
Contributor

jpata commented May 17, 2022

EDIT: not correct, signature redacted. see below.

+reconstruction

@cmsbuild
Copy link
Contributor

This issue is fully signed and ready to be closed.

@jpata
Copy link
Contributor

jpata commented Aug 1, 2022

It was just pointed out in #38890 that this issue is not solved. In fact

Fixed in #31985

violates causality, as this issue was opened to address the remaining bits from the PR. My bad for not noticing.

@jpata
Copy link
Contributor

jpata commented Aug 1, 2022

@cmsbuild please reopen

@qliphy
Copy link
Contributor

qliphy commented Aug 1, 2022

assign l1
remove above l1 sign to trigger the review

@cms-sw cms-sw deleted a comment from cecilecaillol Aug 1, 2022
@cecilecaillol
Copy link
Contributor

@jpata I think this issue is solved by #39169. Can we close the issue?

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

No branches or pull requests

5 participants