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
Add PixelBaryCentreAnalyzer in Alignment offline validation tool #32601
Add PixelBaryCentreAnalyzer in Alignment offline validation tool #32601
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32601/20638
|
A new Pull Request was created by @tocheng (Tongguang) for master. It involves the following packages: Alignment/OfflineValidation @cmsbuild, @pohsun, @yuanchao, @christopheralanwest, @tlampen can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a64171/11962/summary.html Comparison SummarySummary:
|
Hi @tocheng,
|
@@ -0,0 +1,117 @@ | |||
#ifndef Alignment_OfflineValidation_PixelBaryCentreAnalyzer_H |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cmssw
plugins should not have header files, but be fully contained in the .cc file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
const SiPixelQuality* badPixelInfo = &iSetup.getData(siPixelQualityToken_); | ||
|
||
// global position | ||
edm::ESHandle<Alignments> globalAlignments; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing esConsumes
migration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
PixelBaryCentreAnalyzer::initBC(); | ||
|
||
// Get TkAlign from EventSetup: | ||
edm::ESHandle<Alignments> alignments; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing esConsumes
migration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
PixelBaryCentreAnalyzer::initBS(); | ||
|
||
// Get BeamSpot from EventSetup | ||
edm::ESHandle<BeamSpotObjects> beamhandle; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing esConsumes
migration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
#include "FWCore/Framework/interface/Event.h" | ||
#include "FWCore/Framework/interface/MakerMacros.h" | ||
#include "FWCore/Framework/interface/EventSetup.h" | ||
#include "FWCore/Framework/interface/ESHandle.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I got compile error removing Event.h header
"invalid use of incomplete type 'const class edm::Event'
212 | ls_ = iEvent.id().luminosityBlock()"
and compile error removing MakerMacros header
aggregate 'edm::ParameterSetDescription desc' has incomplete type and cannot be defined
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In case it was not clear it's just
#include "FWCore/Framework/interface/ESHandle.h
that is unnecessary after the esConsumes migration.
Hi @tocheng , |
bc8036e
to
6933359
Compare
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32601/20685
|
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32601/20906 |
@cmsbuild, please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a64171/12574/summary.html Comparison SummarySummary:
|
+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 now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
PixelBaryCentreAnalyzer is an ED Analyzer that reads global tracker position from GlobalTag and tracker alignment constants from different ESsources, calculates and stores barycentres of different pixel detector substructure in ROOT TTrees, one for each ESsource.
The One module is used because the analyzer is expected to read (empty event) data in orders of lumi section/run number. (Concurrent lumi section processing is not very needed here).
Plotting script is added to compare barycentres from Prompt Reco, End-of-Year Rereco and so-called Run-2 (Ultra)Legacy Rereco.
PR validation:
Plotting script is used as a validation of the output (Ntuple) from the analyzer
if this PR is a backport please specify the original PR and why you need to backport that PR:
The PR is not a backport.