-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Introduce matrix workflows for Alpaka-based Particle Flow #43701
Conversation
cms-bot internal usage |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-43701/38388
|
A new Pull Request was created by @jsamudio for master. It involves the following packages:
@AdrianoDee, @syuvivida, @antoniovilela, @mandrenguyen, @subirsarkar, @jfernan2, @miquork, @rvenditti, @sunilUIET, @tjavaid, @cmsbuild, @srimanob, @antoniovagnerini, @nothingface0, @fabiocos, @davidlange6, @rappoccio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Hi @jsamudio |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-43701/38389
|
Pull request #43701 was updated. @cmsbuild, @davidlange6, @nothingface0, @rvenditti, @syuvivida, @AdrianoDee, @antoniovilela, @srimanob, @jfernan2, @mandrenguyen, @subirsarkar, @tjavaid, @fabiocos, @miquork, @rappoccio, @sunilUIET, @antoniovagnerini can you please check and sign again. |
Can somebody trigger jenkins? |
enable gpu |
test parameters:
|
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10733b/36811/summary.html Comparison SummarySummary:
GPU Comparison SummarySummary:
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-10733b/37434/summary.html Comparison SummarySummary:
GPU Comparison SummarySummary:
|
+1 |
+pdmv |
+Upgrade |
@cms-sw/dqm-l2 sorry for bothering you explicitly, but I would like to get this PR and its backport merged in time for 14.0.0 tomorrow. Could you confirm that you are aware of it, and are either fine with it, or testing it, or plan to do it soon ? |
+1
|
+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 be automatically merged. |
PR description:
This PR adds new workflows to test Alpaka-based PF modules in CMSSW. Introduced are:
.431
: HCAL Only Reconstruction using Alpaka-based PF, slimmed for benchmarking.432
: HCAL Only Reconstruction on GPU using Alpaka-based PF, with DQM and Validation.433
: HCAL Only Reconstruction on CPU using Alpaka-based PF, with DQM, Validation, and Alpaka vs Legacy PF Cluster comparison.434
: HCAL Only Reconstruction on GPU using Alpaka-based PF, with DQM, Validation, and Alpaka vs Legacy PF Cluster comparison.435
: HCAL Only Reconstruction on GPU using Alpaka-based PF (forced CUDA), with DQM, Validation, and Alpaka vs Legacy PF Cluster comparison0.422
: Alpaka, HCAL only, portable0.423
: Alpaka, HCAL only, portable vs CPU validation0.424
: Alpaka, HCAL only, portable profiling0.492
Alpaka, Pixel quadruplets, Full-RecoPR validation:
All new workflows tested and passing on
12434.
PF cluster Alpaka vs Legacy validation plots from
12434.423
are available at https://hep.baylor.edu/jsamudio/workflowValidation/Usual
runTheMatrix.py -l limited -i all --ibeos
check is done and passes,except forstep3
failure on1000.0
fromFatal Root Error: @SUB=TStorageFactorySystem::Unlink
which I believe is unrelated to the changes in this PR.@hatakeyamak @fwyzard