Skip to content

[ntuple] Add RNTupleProcessor::RequestField by typename string - #22403

Merged
enirolf merged 1 commit into
root-project:masterfrom
enirolf:ntuple-proc-request-field-typename
Jun 9, 2026
Merged

[ntuple] Add RNTupleProcessor::RequestField by typename string#22403
enirolf merged 1 commit into
root-project:masterfrom
enirolf:ntuple-proc-request-field-typename

Conversation

@enirolf

@enirolf enirolf commented May 26, 2026

Copy link
Copy Markdown
Contributor

Adds a non-templated overload of RNTupleProcessor::RequestField that takes a typename string, in case the type is not known at compile time.

@enirolf enirolf self-assigned this May 26, 2026
@enirolf
enirolf requested a review from jblomer as a code owner May 26, 2026 07:56
Comment thread tree/ntuple/test/ntuple_processor.cxx
@github-actions

Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 13h 43m 15s ⏱️
 3 862 tests  3 861 ✅ 0 💤 1 ❌
77 105 runs  77 104 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 997ccde.

@enirolf
enirolf force-pushed the ntuple-proc-request-field-typename branch from 997ccde to 4e16836 Compare May 26, 2026 11:06
@enirolf
enirolf requested a review from pcanal May 26, 2026 11:07
@enirolf
enirolf force-pushed the ntuple-proc-request-field-typename branch from 4e16836 to bdcb912 Compare May 26, 2026 11:17
Comment thread tree/ntuple/test/ntuple_processor.cxx

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

Nice thanks!

@enirolf
enirolf requested a review from pcanal June 8, 2026 06:38
@enirolf
enirolf merged commit 77cf282 into root-project:master Jun 9, 2026
@enirolf
enirolf deleted the ntuple-proc-request-field-typename branch June 16, 2026 07:41
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.

4 participants