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
CalibCalorimetry/EcalLaserAnalyzer: uncomment ClassDef #11787
Conversation
Having classes based on `TObject` without `ClassDef` creates issues while creating a dictionary. According to CVS these files were introduced 6 years and 4 months ago and already had `ClassDef` commented out. If we do not need to store (I/O) then we need to set `ClassVersionID` to **0**. CVS history: http://cvs.web.cern.ch/cvs/cgi-bin/viewcvs.cgi/CMSSW/CalibCalorimetry/EcalLaserAnalyzer/interface/ ROOT `ClassDef` documentation: https://root.cern.ch/root/Using.html Signed-off-by: David Abdurachmanov <David.Abdurachmanov@cern.ch>
Basically with is RFC. It's up to L2 to decide if these classes requires IO or not, if not we probably don't need this dictionary (?). |
A new Pull Request was created by @davidlt for CMSSW_7_6_X. CalibCalorimetry/EcalLaserAnalyzer: uncomment ClassDef It involves the following packages: CalibCalorimetry/EcalLaserAnalyzer @cmsbuild, @mmusich, @diguida, @franzoni, @cerminar can you please review it and eventually sign? Thanks. |
please test |
The tests are being triggered in jenkins. |
-1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
please test |
The tests are being triggered in jenkins. |
Before this is merged I would like to see a statement regarding ClassVersionID. Do we need object I/O or not. |
-1 The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic: |
please test |
The tests are being triggered in jenkins. |
Could we resolve this, this is blocking building with ROOT 6.04 / 6.06. So, do we need object I/O here or not? If yes, then this PR stays as-is, I guess. If not, I will create a different PR. |
ping^1 |
Closed for CMSSW_7_6_X, look at PR for CMSSW_8_0_X. |
Having classes based on
TObject
withoutClassDef
creates issues whilecreating a dictionary. According to CVS these files were introduced 6
years and 4 months ago and already had
ClassDef
commented out. If wedo not need to store (I/O) then we need to set
ClassVersionID
to 0.CVS history: http://cvs.web.cern.ch/cvs/cgi-bin/viewcvs.cgi/CMSSW/CalibCalorimetry/EcalLaserAnalyzer/interface/
ROOT
ClassDef
documentation: https://root.cern.ch/root/Using.htmlSigned-off-by: David Abdurachmanov David.Abdurachmanov@cern.ch