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

LumiProducer blocks concurrent lumi processing #32518

Closed
jpata opened this issue Dec 17, 2020 · 7 comments
Closed

LumiProducer blocks concurrent lumi processing #32518

jpata opened this issue Dec 17, 2020 · 7 comments

Comments

@jpata
Copy link
Contributor

jpata commented Dec 17, 2020

LumiProducer seems to have several issues:

  1. it blocks concurrent lumi processing introduced in 11_3_X: Modules prohibiting concurrent lumis in RelVal workflows #25090 (comment)
  2. it requires updates LumiProducer and LumiCorrectionSource should use SiteLocalConfig service #28608
  3. it seems it is not really used in production at least from 2018: Modules prohibiting concurrent lumis in RelVal workflows #25090 (comment)

cc LUMI POG conveners @gkrintir and Christoph Schwick (I couldn't find the github now), can you clarify what is the status of this module? Is it being used (or foreseen to be used in Run 3) for physics? Given the points above, especially (1), it may be a prudent time to remove this from the sequences, if not used. Otherwise, resource will have to be invested to make sure this module is updated.

@jpata
Copy link
Contributor Author

jpata commented Dec 17, 2020

assign reconstruction

@cmsbuild
Copy link
Contributor

New categories assigned: reconstruction

@slava77,@perrotta,@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 @jpata Joosep Pata.

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

cms-bot commands are listed here

@gkrintir
Copy link
Contributor

thanks @jpata for reporting

indeed it can be removed from the sequences @capalmer85 please verify

many thx all

@plujan
Copy link
Contributor

plujan commented Dec 19, 2020

Yes, I can confirm that this was not used for Run 2 and we have no plans to use it for Run 3, so feel free to take whatever actions are necessary.

@slava77
Copy link
Contributor

slava77 commented Jul 22, 2021

+reconstruction

removed in #33024

@cmsbuild
Copy link
Contributor

This issue is fully signed and ready to be closed.

@qliphy qliphy closed this as completed Jul 23, 2021
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

6 participants