-
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
SiStrip Payload Inspector: refinements to SiStripCondObjectRepresent
#37492
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37492/29194
|
A new Pull Request was created by @mmusich (Marco Musich) for master. It involves the following packages:
@cmsbuild, @ggovi, @tvami, @malbouis, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild , please test |
-1 Failed Tests: Build ClangBuild BuildI found compilation error when building: >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_4_X_2022-04-07-1100/src/CondTools/SiStrip/plugins/SiStripMiscalibrateHelper.cc >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_4_X_2022-04-07-1100/src/CondTools/SiStrip/plugins/SiStripNoisesBuilder.cc >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_4_X_2022-04-07-1100/src/CondTools/SiStrip/plugins/SiStripNoisesFromDBMiscalibrator.cc In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_4_X_2022-04-07-1100/src/CondCore/SiStripPlugins/interface/SiStripPayloadInspectorHelper.h:13, from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_4_X_2022-04-07-1100/src/CondTools/SiStrip/plugins/SiStripMiscalibrateHelper.cc:5: /cvmfs/cms-ib.cern.ch/week1/slc7_amd64_gcc10/cms/cmssw/CMSSW_12_4_X_2022-04-07-1100/src/CondCore/Utilities/interface/PayloadInspector.h:18:10: fatal error: pybind11/pybind11.h: No such file or directory 18 | #include | ^~~~~~~~~~~~~~~~~~~~~ compilation terminated. >> Compiling edm plugin /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_4_X_2022-04-07-1100/src/CondTools/SiStrip/plugins/SiStripNoisesReader.cc gmake: *** [tmp/slc7_amd64_gcc10/src/CondTools/SiStrip/plugins/CondToolsSiStripPlugins/SiStripMiscalibrateHelper.cc.o] Error 1 Clang BuildI found compilation warning while trying to compile with clang. Command used:
See details on the summary page. |
…ndObjectRepresent class
…ipCondObjectRepresent and update scripts
d21c579
to
82449e4
Compare
So, I removed the clang warnings reported here in the last push, but I am not able to reproduce the compilation failure in my checkout using: scram b vclean && BUILD_LOG=yes USER_CXXFLAGS=-DUSE_CMS_DEPRECATED scram b -k -j 16 so I am not sure if triggering the test would result again in a failure. |
@cmsbuild , please test |
There is no such file in release (see: https://github.com/cms-sw/cmssw/tree/master/CondCore/SiStripPlugins), shall I add it? |
ah Ok, sorry I did not check that @mmusich . In that case what is the purpose of
I personally would prefer to merge the packages |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-37492/29208
|
as they are supposed to carry out separate tasks, I would rather not merge them, I went for the recipe you recommended at #37492 (comment) |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-9e5910/23765/summary.html Comparison SummarySummary:
|
+db
|
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
The purpose of this PR is to introduce several improvements and clean-ups into the SiStrip payload inspector classes and builds on top of #37265 and #37333:
THStack
plot for the gains broken down by subdetector.TrackerMaps
for theSiStripDetVOff
object by using class templatesSiStripCondObjectRepresent
classes.PR validation:
The touched classes were tested privately with e.g. the following commands:
getPayloadData.py --plugin pluginSiStripNoises_PayloadInspector --plot plot_SiStripNoiseCompareByPartition --tag SiStripNoise_v2_prompt --tagtwo SiStripNoise_v2_prompt --time_type Run --iovs '{"start_iov": "348878", "end_iov": "348878"}' --iovstwo '{"start_iov": "349927", "end_iov": "349927"}' --db Prod --test ;
etPayloadData.py --plugin pluginSiStripNoises_PayloadInspector --plot plot_SiStripNoiseDiffByPartition --tag SiStripNoise_v2_prompt --tagtwo SiStripNoise_v2_prompt --time_type Run --iovs '{"start_iov": "348878", "end_iov": "348878"}' --iovstwo '{"start_iov": "349927", "end_iov": "349927"}' --db Prod --test ;
getPayloadData.py --plugin pluginSiStripDetVOff_PayloadInspector --plot plot_SiStripDetVOff_IsModuleVOff_TrackerMap --tag SiStripDetVOff_v6_prompt --time_type Time --iovs '{"start_iov": "7083471724997514240", "end_iov": "7083471724997514240"}' --db Prod --test ;
if this PR is a backport please specify the original PR and why you need to backport that PR:
N/A