Skip to content

Conversation

@guitargeek
Copy link
Contributor

In ecd9863, the class version of the RooAbsArg class was incremented.

As explained in #8791, there are sometimes warnings in the IO of derived classes if their version number is not increased as well.

Increasing the class version of RooRealVar indeed fixes this warning that one gets right now when reading old workspaces:

Warning in <TStreamerInfo::BuildCheck>:
   The StreamerInfo of class RooRealVar read from file toyws/WS-VHbb-STXS_mu_toy_new.root
   has the same version (=8) as the active class but a different checksum.
   You should update the version to ClassDef(RooRealVar,9).
   Do not try to write objects with the current class definition,
   the files will not be readable.

Warning in <TStreamerInfo::CompareContent>: The following data member of
the on-file layout version 8 of class 'RooRealVar' differs from
the in-memory layout version 8:
   RooAbsBinning _binning; //
vs
   unique_ptr<RooAbsBinning,default_delete<RooAbsBinning> > _binning;

In ecd9863, the class version of the `RooAbsArg` class was
incremented.

As explained in root-project#8791, there are sometimes warnings in the IO of derived
classes if their version number is not increased as well.

Increasing the class version of RooRealVar indeed fixes this warning
that one gets right now when reading old workspaces:

```
Warning in <TStreamerInfo::BuildCheck>:
   The StreamerInfo of class RooRealVar read from file toyws/WS-VHbb-STXS_mu_toy_new.root
   has the same version (=8) as the active class but a different checksum.
   You should update the version to ClassDef(RooRealVar,9).
   Do not try to write objects with the current class definition,
   the files will not be readable.

Warning in <TStreamerInfo::CompareContent>: The following data member of
the on-file layout version 8 of class 'RooRealVar' differs from
the in-memory layout version 8:
   RooAbsBinning _binning; //
vs
   unique_ptr<RooAbsBinning,default_delete<RooAbsBinning> > _binning;
```
@phsft-bot
Copy link

Starting build on ROOT-debian10-i386/soversion, ROOT-performance-centos8-multicore/cxx17, ROOT-ubuntu18.04/nortcxxmod, ROOT-ubuntu2004/python3, mac12/noimt, mac11/cxx14, windows10/cxx14
How to customize builds

@phsft-bot
Copy link

Build failed on ROOT-ubuntu18.04/nortcxxmod.
Running on sft-ubuntu-1804-2.cern.ch:/build/workspace/root-pullrequests-build
See console output.

Errors:

  • [2023-01-25T15:01:58.761Z] FAILED: tmva/sofie/test/CMakeFiles/SofieCompileModels_ONNX.util

Copy link
Member

@lmoneta lmoneta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@guitargeek guitargeek merged commit bce142c into root-project:master Jan 25, 2023
@guitargeek guitargeek deleted the roorealvar_1 branch January 25, 2023 20:06
@phsft-bot
Copy link

Build failed on windows10/cxx14.
Running on null:C:\build\workspace\root-pullrequests-build
See console output.

guitargeek added a commit to guitargeek/root that referenced this pull request Jan 24, 2025
guitargeek added a commit that referenced this pull request Jan 24, 2025
Same reasoning as in commit bce142c from #12114.

Closes #17163.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants