-
Notifications
You must be signed in to change notification settings - Fork 8
add calibration-to-surrounding cell offset map #3
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
Conversation
|
A new Pull Request was created by @RSalvatico for branch main. @Dr15Jones, @Moanwar, @bsunanda, @civanch, @cmsbuild, @iarspider, @kpedro88, @makortel, @mdhildreth, @smuzaffar, @srimanob, @subirsarkar can you please review it and eventually sign? Thanks. |
|
cms-bot internal usage |
|
please test |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2a8261/45473/summary.html Comparison SummarySummary:
|
|
Pull request #3 was updated. |
|
please test |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2a8261/45915/summary.html Comparison SummarySummary:
|
move offset file to CellMaps directory add sequence entry to the table
|
Pull request #3 was updated. |
|
please test |
|
-1 Failed Tests: Build BuildI found compilation error when building: >> Subsystem HLTrigger built >> Subsystem Utilities built Copying tmp/el8_amd64_gcc12/src/RecoTracker/LSTCore/src/alpaka/RecoTrackerLSTCoreROCmAsync/libRecoTrackerLSTCoreROCmAsync_rocm.a to productstore area: cp: cannot stat 'tmp/el8_amd64_gcc12/src/RecoTracker/LSTCore/src/alpaka/RecoTrackerLSTCoreROCmAsync/libRecoTrackerLSTCoreROCmAsync_rocm.a': No such file or directory >> Deleted: tmp/el8_amd64_gcc12/src/RecoTracker/LSTCore/src/alpaka/RecoTrackerLSTCoreROCmAsync/libRecoTrackerLSTCoreROCmAsync_rocm.a gmake: *** [config/SCRAM/GMake/Makefile.rules:1886: tmp/el8_amd64_gcc12/src/RecoTracker/LSTCore/src/alpaka/RecoTrackerLSTCoreROCmAsync/libRecoTrackerLSTCoreROCmAsync_rocm.a] Error 1 Copying tmp/el8_amd64_gcc12/src/RecoTracker/LSTCore/src/alpaka/RecoTrackerLSTCoreCudaAsync/libRecoTrackerLSTCoreCudaAsync_nv.a to productstore area: cp: cannot stat 'tmp/el8_amd64_gcc12/src/RecoTracker/LSTCore/src/alpaka/RecoTrackerLSTCoreCudaAsync/libRecoTrackerLSTCoreCudaAsync_nv.a': No such file or directory >> Deleted: tmp/el8_amd64_gcc12/src/RecoTracker/LSTCore/src/alpaka/RecoTrackerLSTCoreCudaAsync/libRecoTrackerLSTCoreCudaAsync_nv.a gmake: *** [config/SCRAM/GMake/Makefile.rules:1886: tmp/el8_amd64_gcc12/src/RecoTracker/LSTCore/src/alpaka/RecoTrackerLSTCoreCudaAsync/libRecoTrackerLSTCoreCudaAsync_nv.a] Error 1 >> Entering Package Configuration/Skimming |
|
Not sure what was wrong here, so I'll just try to trigger the tests again |
|
please test |
The last PR tests used an IB where |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-2a8261/46048/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:
|
|
Hello. The txt file added by this PR is needed for this cms-sw PR to work properly. It won't break any "central" workflow if it's not integrated before the cms-sw PR, but there are some HGCAL testing workflows that won't work out of the box if it doesn't happen. So it would be good to get signatures or comments. Thanks! |
|
Hi all, is there anything needed before this PR can be merged? (Apologies if I'm being pushy, but this would facilitate HGCal development.) |
|
Hi, as cms-sw/cmssw#47829 is merged, can this be merged as well please? |
|
+Upgrade |
|
Hello, |
|
+externals |
PR description
Add HGCAL calibration-to-surrounding cell offset map, based on the tables in this document.