Skip to content

[rfile] Add RFile tutorials for RNTuple and TTree - #23688

Merged
silverweed merged 4 commits into
root-project:masterfrom
silverweed:rfile_ttree
Oct 20, 2026
Merged

silverweed merged 4 commits into
root-project:masterfrom
silverweed:rfile_ttree

Conversation

@silverweed

Copy link
Copy Markdown
Contributor

Also move rfile tutorials to a dedicated directory

@vepadulano vepadulano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! LGTM, see two minor comments.

Comment thread tutorials/io/experimental/rfile/rfile002_rntuple.py Outdated
Comment thread tutorials/io/experimental/rfile/rfile003_rfile.py Outdated
@github-actions

github-actions Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Test Results

    24 files      24 suites   4d 2h 45m 40s ⏱️
 3 889 tests  3 887 ✅ 0 💤 2 ❌
83 283 runs  83 281 ✅ 0 💤 2 ❌

For more details on these failures, see this check.

Results for commit b6d1a76.

♻️ This comment has been updated with latest results.

@silverweed

Copy link
Copy Markdown
Contributor Author

@vepadulano I applied your suggestions and also fixed the way ROOT.Experimental.RNTupleWriter_Append was used (now it's hidden in the pythonization and properly supports the context manager).

@vepadulano vepadulano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! A few more comments from my side.

Comment thread bindings/pyroot/pythonizations/python/ROOT/_pythonization/_rntuple.py Outdated
Comment thread tutorials/io/experimental/rfile/rfile003_ttree.C
Comment thread tutorials/io/experimental/rfile/rfile003_ttree.py Outdated
Comment thread tutorials/io/experimental/rfile/rfile002_rntuple.C
@silverweed
silverweed force-pushed the rfile_ttree branch 2 times, most recently from 696433c to 84b670e Compare October 13, 2026 09:11
Comment thread bindings/pyroot/pythonizations/python/ROOT/_pythonization/_rntuple.py Outdated

@jblomer jblomer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM but see CI failure

@silverweed silverweed added the clean build Ask CI to do non-incremental build on PR label Oct 13, 2026

@vepadulano vepadulano left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

Comment thread tutorials/io/experimental/rfile/rfile002_rntuple.C
@silverweed
silverweed force-pushed the rfile_ttree branch 2 times, most recently from ff3cf46 to 5080b27 Compare October 16, 2026 07:21
@silverweed
silverweed requested a review from bellenot as a code owner October 19, 2026 13:17
@silverweed
silverweed merged commit d87765f into root-project:master Oct 20, 2026
32 of 36 checks passed
@silverweed
silverweed deleted the rfile_ttree branch October 20, 2026 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clean build Ask CI to do non-incremental build on PR in:I/O in:Tutorials

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants