-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Reverting the Phase-2 HLT menu to not use Tasks
#42940
Comments
assign hlt |
New categories assigned: hlt @Martin-Grunewald,@mmusich,@missirol you have been requested to review this Pull request/Issue and eventually sign? Thanks |
A new Issue was created by @mmusich Marco Musich. @rappoccio, @antoniovilela, @smuzaffar, @Dr15Jones, @sextonkennedy, @makortel can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
Just to bring up one option (now that I have some brain bandwidth again), following the discussion in #36938, a There is one caveat though: the modules in If organizing the modules in |
#43296 implements the printout of ConditionalTask modules that are not consumed by any module in any of the Paths the ConditionalTask (in question) is associated with, as discussed with some of the HLT folks. |
IIUC #44025 starts to implement things in this direction. |
#44604 completes the migration of the phase-2 HLT menu to use sequences. |
+hlt
|
This issue is fully signed and ready to be closed. |
@cmsbuild, please close |
In #42934 (comment) it was suggested to consider reverting the Phase-2 HLT menu to not use
Tasks
to avoid triggering undesired unscheduled execution ofEDProducer
s.The 14_0_X cycle was agreed to be a reasonable target to achieve this (see #42934 (comment)).
The text was updated successfully, but these errors were encountered: