Skip to content

[RF] Remove accidentally introduced I/O capabilities of RooMinimizer. #8652

@hageboeck

Description

@hageboeck

@guitargeek @egpbos

Stop, stop!! e3b73a8 needs to be reverted before releasing ROOT.

ClassDef(..., 0) means: Don't support I/O.
In e3b73a8, you switched on I/O capabilities, so please revert! When people start writing minimiser states to files, you will have to support deserialisation in the future. It's better to not support that.

That by the way also holds for any classes the are introduced in the MP-PRs. Please consider well wether they are allowed to support I/O or not.

Originally posted by @hageboeck in #8569 (comment)

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions