-
Notifications
You must be signed in to change notification settings - Fork 2
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
update data used for the Pixel,ECal,HCal GPU clients for post-alpaka migration in the HLT menu #7
Conversation
…migration in the HLT menu
A new Pull Request was created by @mmusich for branch main. @nothingface0, @aandvalenzuela, @tjavaid, @cmsbuild, @smuzaffar, @syuvivida, @iarspider, @rvenditti, @antoniovagnerini can you please review it and eventually sign? Thanks. |
cms-bot internal usage |
test parameters: |
@cmsbuild, please test |
-1 Failed Tests: RelVals-INPUT The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: RelVals-INPUT
Comparison SummarySummary:
|
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9465b4/39400/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
REMINDER @sextonkennedy, @antoniovilela, @rappoccio: This PR was tested with cms-sw/cmssw#44971, please check if they should be merged together |
+1 |
+1 |
merge |
After the migration of the HLT menu in 2024 to use
alpaka
-based modules for the local reconstruction of the Pixel, Ecal at HLT (see TSG tickets CMSHLT-3109, CMSHLT-3125) the naming of the collections in output to theDQMGPUvsCPU
stream has been updated (see TSG ticket CMSHLT-3132).This has prompted
cmssw
PRs in order to comply with the changes:siPixelPhase1RawDataErrorComparator
collections post-alpaka migration cms-sw/cmssw#44934 (for Pixel)Due to that, the data used in unit tests currently running in IB and PR tests is out-of-synch with the collection names requested by the clients, see e.g. these logs.
The purpose of this PR (joint to the companion cmssw PR cms-sw/cmssw#44971) is to update the data used for the tests to a fresher set coming from run380649 (from Run2024D pp run OMS link, which features the new collection names.