Add Phase 2 HLT CPU vs. GPU wf to the ph2_hlt PR tests#50505
Conversation
|
cms-bot internal usage |
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50505/48673
|
|
A new Pull Request was created by @VourMa for master. It involves the following packages:
@AdrianoDee, @DickyChant, @antoniovagnerini, @cmsbuild, @miquork can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
| 'ph2_hlt' : [prefixDet+34.75, # HLT phase-2 timing menu | ||
| prefixDet+34.7501, # HLT phase-2 tracking-only menu | ||
| prefixDet+34.7502, # HLT phase-2 tracking menu with tracking ntuple | ||
| prefixDet+34.7503, # HLT phase-2 menu, CPU vs. GPU validation |
There was a problem hiding this comment.
I am not sure this change is appropriate, in general we don't run these tests with the "gpu" enabled.
There was a problem hiding this comment.
I feel like there are quite a few cases where it happens that we run the ph2_hlt tests with GPUs enabled, so this is targeting those cases as an extra tests. However, I agree it is situational.
If you think this is unnecessary, I will close the PR. Let me know.
|
test parameters:
|
|
please test |
|
+1 Size: This PR adds an extra 28KB to repository Comparison SummarySummary:
|
|
Milestone for this pull request has been moved to CMSSW_17_0_X. Please open a backport if it should also go in to CMSSW_16_1_X. |
|
please test |
|
-1 Failed Tests: RelVals Failed RelVals
Expand to see more relval errors ... |
|
please test |
|
+1 Size: This PR adds an extra 16KB to repository Comparison SummarySummary:
|
|
+pdmv |
|
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. @sextonkennedy, @mandrenguyen, @ftenchini (and backports should be raised in the release meeting by the corresponding L2) |
|
+1 |
Small follow up to #50336, adding the Phase 2 HLT CPU vs. GPU workflow to the list of
ph2_hltPR tests. To be tested during PR tests.This PR can also be used to test cms-sw/cms-bot#2710.