-
Notifications
You must be signed in to change notification settings - Fork 20
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
More TrackerAlignment ES get() calls #48
Comments
Hi @bbilin I found some more ES get()-s for your enjoyment :) |
Alignment/TrackerAlignment
ES get()
Hi @bbilin did you see this? |
Hi @tvami yes, I saw it :) I was just too busy to come to this yet, but will do so later this week. Thanks for repinging. |
Hi @bbilin , this is my weekly kind ping :) |
will ping you back as soon as I have some implementations- currently I had no time to look at it yet. Will let you know asap. |
Resolved in cms-sw/cmssw#36193 |
Alignment/TrackerAlignment
https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc7_amd64_gcc900/CMSSW_12_2_CMSDEPRECATED_X_2021-11-03-2300/Alignment/TrackerAlignment
crtl-f for
.get(
finds 12 more occurrences of deprecated ES get callsthen going to
Alignment/CommonAlignmentProducer
https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc7_amd64_gcc900/CMSSW_12_2_CMSDEPRECATED_X_2021-11-03-2300/Alignment/CommonAlignmentProducer
I find 8 occurrences
Alignment/LaserAlignmentSimulation
src/Alignment/LaserAlignmentSimulation/test/SimAnalyzer.cc:255:67: warning: 'bool edm::eventsetup::EventSetupRecord::get(HolderT&) const [with HolderT = edm::ESHandle]' is deprecated [-Wdeprecated-declarations]
255 | theSetup.get().get(theTrackerGeometry);
Alignment/SurveyAnalysis
https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc7_amd64_gcc900/CMSSW_12_2_CMSDEPRECATED_X_2021-11-03-2300/Alignment/SurveyAnalysis
has 16 occurrences
The text was updated successfully, but these errors were encountered: