Skip to content

[ntuple] Store SoA field streamer info#22114

Open
jblomer wants to merge 7 commits intoroot-project:masterfrom
jblomer:ntuple-fix-soa-streamerinfo
Open

[ntuple] Store SoA field streamer info#22114
jblomer wants to merge 7 commits intoroot-project:masterfrom
jblomer:ntuple-fix-soa-streamerinfo

Conversation

@jblomer
Copy link
Copy Markdown
Contributor

@jblomer jblomer commented Apr 30, 2026

Triggers also storing the class checksum and class version on disk. Also ensure that SoA class version and underlying record type version are identical (for the time being). To be backported to 6.40.

@jblomer jblomer self-assigned this Apr 30, 2026
@jblomer jblomer force-pushed the ntuple-fix-soa-streamerinfo branch from f40f6b9 to 0d516b3 Compare April 30, 2026 12:24
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 30, 2026

Test Results

    21 files      21 suites   3d 5h 27m 38s ⏱️
 3 853 tests  3 852 ✅ 0 💤 1 ❌
72 332 runs  72 331 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 0886124.

♻️ This comment has been updated with latest results.

Comment thread tree/ntuple/test/ntuple_soa.cxx Outdated
Copy link
Copy Markdown
Member

@pcanal pcanal left a comment

Choose a reason for hiding this comment

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

LGTM.

While currently not needed, if we want to execute I/O customization
rules on the SoA class, we need the corresponding streamer info on file.
@jblomer jblomer force-pushed the ntuple-fix-soa-streamerinfo branch from 0d516b3 to 0886124 Compare April 30, 2026 19:26
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.

2 participants