-
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
Removing CUDA/gpu
from Pixel code configs and dropping all CUDA wfs
#46853
base: master
Are you sure you want to change the base?
Removing CUDA/gpu
from Pixel code configs and dropping all CUDA wfs
#46853
Conversation
cms-bot internal usage |
Well, apparently already these changes involve many parties. So maybe I'll push the drop of the |
enable gpu |
please test |
A new Pull Request was created by @AdrianoDee for master. It involves the following packages:
@AdrianoDee, @Moanwar, @antoniovilela, @atpathak, @consuegs, @davidlange6, @DickyChant, @fabiocos, @jfernan2, @mandrenguyen, @miquork, @perrotta, @rappoccio, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
assign heterogeneous |
5d626de
to
77de940
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46853/42956
|
Pull request #46853 was updated. @AdrianoDee, @Moanwar, @antoniovagnerini, @antoniovilela, @atpathak, @consuegs, @davidlange6, @DickyChant, @fabiocos, @fwyzard, @jfernan2, @makortel, @mandrenguyen, @miquork, @perrotta, @rappoccio, @rseidita, @srimanob, @subirsarkar can you please check and sign again. |
-1 Failed Tests: UnitTests Unit TestsI found 1 errors in the following unit tests: ---> test TestDQMOnlineClients-compilation_hi_run had ERRORS Comparison SummarySummary:
GPU Comparison SummarySummary:
|
72e2818
to
8cf7a09
Compare
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-46853/42959
|
Pull request #46853 was updated. @AdrianoDee, @Moanwar, @antoniovagnerini, @antoniovilela, @atpathak, @consuegs, @davidlange6, @DickyChant, @fabiocos, @fwyzard, @jfernan2, @makortel, @mandrenguyen, @miquork, @perrotta, @rappoccio, @rseidita, @srimanob, @subirsarkar can you please check and sign again. |
+1 Size: This PR adds an extra 20KB to repository
Comparison SummarySummary:
GPU Comparison SummarySummary:
|
PR description:
This PR proposes:
pixelNtupletFit_cff
modifier.A subsequent step would be to remove the
gpu
modifier, but since this involves code from many parties, I prefer to have it separated.