Skip to content

[ntuple] add read support to SoA I/O#22139

Open
jblomer wants to merge 4 commits into
root-project:masterfrom
jblomer:ntuple-soa-read
Open

[ntuple] add read support to SoA I/O#22139
jblomer wants to merge 4 commits into
root-project:masterfrom
jblomer:ntuple-soa-read

Conversation

@jblomer
Copy link
Copy Markdown
Contributor

@jblomer jblomer commented May 4, 2026

Part of #19230

Tutorial output:

************************************ NTUPLE ************************************
* N-Tuple : ntpl                                                               *
* Entries : 4                                                                  *
********************************************************************************
* Field 1           : points (PointSoA)                                        *
*   Field 1.1       : _0 (PointRecord)                                         *
*     Field 1.1.1   : fX (float)                                               *
*     Field 1.1.2   : fY (float)                                               *
********************************************************************************
Approximated PI with 100 points to 3.2
Approximated PI with 500 points to 3.104
Approximated PI with 1000 points to 3.172
Approximated PI with 10000 points to 3.1388
image

@jblomer jblomer self-assigned this May 4, 2026
@jblomer jblomer requested a review from couet as a code owner May 4, 2026 21:54
@jblomer jblomer force-pushed the ntuple-soa-read branch 2 times, most recently from c87697d to dc1881d Compare May 4, 2026 22:01
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

Test Results

    22 files      22 suites   3d 11h 54m 40s ⏱️
 3 863 tests  3 863 ✅ 0 💤 0 ❌
76 218 runs  76 218 ✅ 0 💤 0 ❌

Results for commit 0a58738.

♻️ This comment has been updated with latest results.

@jblomer jblomer force-pushed the ntuple-soa-read branch 2 times, most recently from 0897dc6 to c41fbb0 Compare May 5, 2026 19:22
Copy link
Copy Markdown
Member

@hahnjo hahnjo left a comment

Choose a reason for hiding this comment

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

Very nice! Two minor comments about comments 😉

Comment thread tree/ntuple/inc/ROOT/RField/RFieldSoA.hxx
Comment thread tutorials/io/ntuple/ntpl020_soa.C Outdated
Comment thread tree/ntuple/inc/ROOT/RField/RFieldSoA.hxx Outdated
@jblomer jblomer force-pushed the ntuple-soa-read branch from c41fbb0 to 6a5ee7a Compare May 23, 2026 20:23
@jblomer jblomer force-pushed the ntuple-soa-read branch from 6a5ee7a to 0a58738 Compare May 25, 2026 20:07
@jblomer jblomer requested a review from bellenot as a code owner May 25, 2026 20:07
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