Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Addition of single iteration Patatrack and LST HLT configurations and workflows #46828
Addition of single iteration Patatrack and LST HLT configurations and workflows #46828
Changes from 1 commit
08ed70d
f7f4f00
3514abf
181e137
0f32bd0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as a generic note, I'm proposing to monitor the intermediate track collection produced in the HLT menu at #46823.
I suppose we should customize those developments once this PR enters.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree and, in fact, I have already done this while I was preparing the PR: please check this commit. Especially given the presence of #46823, I thought I would include it in a next PR (after rebasing) but let me know whether it makes sense to include it already and whether you have an comments on the implementation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's do it as a follow-up step (incidentally, I think we need to change the event content of the step in which the HLT runs in order to make the collections available for the step when the DQM is run).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed, in my tests I was manually keeping the extra collections in the
cmsDriver
command. Let's follow up with this after this gets merged.