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
Fixing some edge cases in HLT-Patatrack customisations #34992
Fixing some edge cases in HLT-Patatrack customisations #34992
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34992/24844
|
A new Pull Request was created by @missirol (Marino Missiroli) for master. It involves the following packages:
@Martin-Grunewald, @cmsbuild can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@@ -225,11 +224,10 @@ def customisePixelLocalReconstruction(process): | |||
# customisation for running the "Patatrack" pixel track reconstruction | |||
def customisePixelTrackReconstruction(process): | |||
|
|||
if not 'HLTRecoPixelTracksSequence' in process.__dict__: | |||
if not all(seq in process.__dict__ for seq in ['HLTRecoPixelTracksSequence', 'HLTRecopixelvertexingSequence']): |
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.
do we need/want to support the case where the configuration has HLTRecoPixelTracksSequence
but not HLTRecopixelvertexingSequence
?
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 couldn't find a use case like this in the GRun
menu, but supporting this seems easy enough.
Tentatively added in e397a38176c
.
Checked that e397a38176c
works as intended, by testing a config with
+process.MC_PixelTracksOnly_v0 = cms.Path(process.HLTRecoPixelTracksSequence)
+process.setSchedule_(cms.Schedule(process.MC_PixelTracksOnly_v0))
+process.prune()
#User-defined customization functions
from HLTrigger.Configuration.customizeHLTforPatatrack import customizeHLTforPatatrackTriplets
process = customizeHLTforPatatrackTriplets(process)
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.
Nice, thanks @missirol .
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34992/24849
|
Pull request #34992 was updated. @Martin-Grunewald can you please check and sign again. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-097e4d/17998/summary.html Comparison SummarySummary:
|
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR is an attempt to make the HLT-Patatrack customisations introduced in #34956 a bit more robust (mostly to handle some edge cases, e.g. HLT configs with only a limited number of paths; see this comment for an example).
Suggestions on the implementation are welcomed.
Attn: @fwyzard
PR validation:
With the PR, the customisations are applied as intended in the following 3 tests (without PR, the first 2 tests wouldn't work correctly).
If this PR is a backport, please specify the original PR and why you need to backport that PR:
N/A