Chris Jones

please test

I did a test where I wrote a 10,000 event file containing only `recoPFCandidates_particleFlow__RECO` using standard AOD settings. If I wrote the file with no compression, this PR made the...

@pcanal So I changed the inheritance for `reco::PFCandidate`, dropping one of the intermediate base classes. When I did that I updated all the class versions as well. However, now the...

So I wrote a simple EDAnalyzer which dumps the contents of a `std::vector`. I ran it using both the original code and the new code, both reading the same input...

So `PFCandidate::energy()` is read from `LeafCandidate::m_state:: p4Cartesian_` where that member variable is declared _transient_ to ROOT. https://github.com/cms-sw/cmssw/blob/e52f99a59bb77ceec0102769446b26c321662ae8/DataFormats/Candidate/src/classes_def.xml#L7 the value is supposed to be filled by the IO rule https://github.com/cms-sw/cmssw/blob/e52f99a59bb77ceec0102769446b26c321662ae8/DataFormats/Candidate/src/classes_def.xml#L9-L12 So...

So pT, eta and phi come from `LeafCandidate::m_state::p4Polar_` which is stored.

So if I do ``` [cdj@cmslpc-c8-heavy01 matrix]$ root -l root://eoscms.cern.ch//eos/cms/store/user/cmsbuild/store/data/Run2017F/JetHT/AOD/17Nov2017-v1/70000/0054D144-36DF-E711-B074-02163E01A47B.root root [1] Events->Scan("recoPFCandidates_particleFlow__RECO.obj.m_state.energy()") ``` using the original format I get ``` *********************************** * Row * Instance * recoPFCan * ***********************************...

I'm doing this using CMSSW_12_5_0_pre4 which is using a version of ROOT 6.24.07

please test

please test