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
Apply Naming Rules to BeamSpotOnlineObjects
methods
#36348
Apply Naming Rules to BeamSpotOnlineObjects
methods
#36348
Conversation
code-checks |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36348/27108
|
A new Pull Request was created by @francescobrivio for master. It involves the following packages:
@malbouis, @yuanchao, @pmandrik, @emanueleusai, @ahmad3213, @tvami, @cmsbuild, @ggovi, @jfernan2, @slava77, @jpata, @francescobrivio, @pbo0, @rvenditti can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cmsbuild please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-8817e4/20965/summary.html Comparison SummarySummary:
|
+1 |
Just pointing out for the record that while that is possible here and that's the preferred choice stated in the slides, that's by no means necessary (as the current state of affairs falls in the second category enumerated in the reference slides). |
+1
|
+reconstruction
|
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:
Following the discussion in #35338 (comment) I made this rather technical PR that to update the methods of
BeamSpotOnlineObjects
to follow CMSSW Naming Rules (specifically 2.8/2.9/2.10/2.11) so that:set*
get
I used
git cms-checkdeps -a
to modify all the packages depending on theBeamSpotOnlineObjects
class.A second PR will follow this in order to update also the methods of
BeamSpotObjects
.I also profited of this PR to migrate
CondTools/BeamSpot/plugins/BeamSpotOnlineRecordsWriter.cc
to use an object created on the stack as suggested in this slide and part of cms-AlCaDB/AlCaTools#28.PR validation:
Code compiles.
Backport:
Not a backport, no backport needed.
FYI: @dzuolo @lguzzi @gennai