Skip to content

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

Open
silverweed wants to merge 4 commits into
root-project:masterfrom
silverweed:rfile_ttree
Open

silverweed wants to merge 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

This branch has not been deployed

No deployments
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