Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ntuple] Split RNTupleView<T, bool> in two classes #16321

Closed
jblomer opened this issue Aug 28, 2024 · 17 comments · Fixed by #16556
Closed

[ntuple] Split RNTupleView<T, bool> in two classes #16321

jblomer opened this issue Aug 28, 2024 · 17 comments · Fixed by #16556

Comments

@jblomer
Copy link
Contributor

jblomer commented Aug 28, 2024

Explain what you would like to see improved and how.

The boolean template in RNTupleView<T, bool> steers whether or not the view is owned by the user. It would be more clear to use two different classes.

ROOT version

master

Installation method

n/a

Operating system

n/a

Additional context

No response

Copy link

github-actions bot commented Oct 2, 2024

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

16 similar comments
Copy link

github-actions bot commented Oct 3, 2024

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

github-actions bot commented Oct 4, 2024

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

github-actions bot commented Oct 7, 2024

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

github-actions bot commented Oct 8, 2024

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

github-actions bot commented Oct 9, 2024

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

Copy link

Hi @jblomer, @silverweed, @vepadulano,

It appears this issue is closed, but wasn't yet added to a project. Please add upcoming versions that will include the fix, or 'not applicable' otherwise.

Sincerely,
🤖

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