Skip to content

[df] Register Hist() pythonization only if available#22041

Merged
hahnjo merged 1 commit intoroot-project:masterfrom
hahnjo:hist-df-pythonization-fix
Apr 24, 2026
Merged

[df] Register Hist() pythonization only if available#22041
hahnjo merged 1 commit intoroot-project:masterfrom
hahnjo:hist-df-pythonization-fix

Conversation

@hahnjo
Copy link
Copy Markdown
Member

@hahnjo hahnjo commented Apr 24, 2026

If ROOT is built without the root7 option, the method doesn't exist which leads to errors when registering the pythonization.

Fixes commit 295345b ("[df] Pythonize Hist() with list of axes")

If ROOT is built without the root7 option, the method doesn't exist
which leads to errors when registering the pythonization.

Fixes commit 295345b ("[df] Pythonize Hist() with list of axes")
@hahnjo hahnjo self-assigned this Apr 24, 2026
@hahnjo hahnjo changed the title [df] Register Hist() pythonization only if available [df] Register Hist() pythonization only if available Apr 24, 2026
Copy link
Copy Markdown
Contributor

@siliataider siliataider left a comment

Choose a reason for hiding this comment

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

Thanks!

@github-actions
Copy link
Copy Markdown

Test Results

    22 files      22 suites   3d 7h 43m 2s ⏱️
 3 847 tests  3 842 ✅   1 💤 4 ❌
75 965 runs  75 825 ✅ 136 💤 4 ❌

For more details on these failures, see this check.

Results for commit c03c180.

@hahnjo hahnjo merged commit c30f543 into root-project:master Apr 24, 2026
33 of 48 checks passed
@hahnjo hahnjo deleted the hist-df-pythonization-fix branch April 24, 2026 11:28
@hahnjo
Copy link
Copy Markdown
Member Author

hahnjo commented Apr 24, 2026

/backport to 6.40

@root-project-bot
Copy link
Copy Markdown

Something went wrong with the backport to 6.40: @hahnjo please see the logs

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.

5 participants