Skip to content

[RDF][Docs] Group the RInterface API in doxygen into Actions / Transformations / General#22320

Open
hageboeck wants to merge 2 commits into
root-project:masterfrom
hageboeck:rdf_interfaceGrouping
Open

[RDF][Docs] Group the RInterface API in doxygen into Actions / Transformations / General#22320
hageboeck wants to merge 2 commits into
root-project:masterfrom
hageboeck:rdf_interfaceGrouping

Conversation

@hageboeck

@hageboeck hageboeck commented May 18, 2026

Copy link
Copy Markdown
Member

Group the RDataFrame interface in the doxygen overview. This results in a structure like this:

| image | image | image |

@hageboeck hageboeck self-assigned this May 18, 2026
@github-actions

github-actions Bot commented May 18, 2026

Copy link
Copy Markdown

Test Results

    20 files      20 suites   3d 5h 36m 37s ⏱️
 3 855 tests  3 819 ✅ 0 💤 36 ❌
69 833 runs  69 785 ✅ 1 💤 47 ❌

For more details on these failures, see this check.

Results for commit f07dc90.

♻️ This comment has been updated with latest results.

@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.

Very nice! I have left some comments only related to the last commit.

Comment thread tree/dataframe/inc/ROOT/RDF/RInterface.hxx
Comment thread tree/dataframe/inc/ROOT/RDF/RInterface.hxx
Comment thread tree/dataframe/inc/ROOT/RDF/RInterface.hxx
Comment thread tree/dataframe/inc/ROOT/RDF/RInterface.hxx Outdated
Comment thread tree/dataframe/inc/ROOT/RDF/RInterface.hxx Outdated
Comment thread tree/dataframe/inc/ROOT/RDF/RInterface.hxx Outdated
@hageboeck hageboeck force-pushed the rdf_interfaceGrouping branch from 902b341 to 2c7c27e Compare May 20, 2026 10:57
Create groups for transformations and actions. This makes it easier to
look at the RDF interface at a glance, because base-class functions are
inlined in the list of functions.
This required reordering of the code, but this is an NFC.
@hageboeck hageboeck force-pushed the rdf_interfaceGrouping branch from 452f04d to 2fe579f Compare May 20, 2026 11:20
@hageboeck hageboeck marked this pull request as ready for review May 20, 2026 11:33
@hageboeck hageboeck requested a review from martamaja10 as a code owner May 20, 2026 11:33
@hageboeck hageboeck force-pushed the rdf_interfaceGrouping branch from 2fe579f to f07dc90 Compare May 20, 2026 11:37

@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.

Very nice thanks! Let's wait for the CI to finish before merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants