Skip to content

[build] Enable Davix in Python wheels #20360

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.

Sign up for GitHub

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

Jump to bottom
Open
vepadulano wants to merge 6 commits into
base: master
Choose a base branch
from
Open

[build] Enable Davix in Python wheels #20360

vepadulano wants to merge 6 commits into from
+11 −11

Conversation

@vepadulano
Copy link
Member

@vepadulano vepadulano commented Nov 9, 2025

No description provided.

@vepadulano vepadulano self-assigned this Nov 9, 2025
@vepadulano vepadulano requested a review from dpiparo as a code owner November 9, 2025 17:44
@vepadulano vepadulano added skip ci Skip the full builds on the actions runners skip code analysis Skip the code analysis CI steps for this PR, including verifying clang-formatting and running Ruff. build-python-wheels Run the workflow to build Python wheels and removed build-python-wheels Run the workflow to build Python wheels labels Nov 9, 2025
@github-actions
Copy link

github-actions bot commented Nov 9, 2025 •
edited
Loading

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit fc0c69d.

♻️ This comment has been updated with latest results.

@vepadulano vepadulano added the build-python-wheels Run the workflow to build Python wheels label Nov 9, 2025
guitargeek
guitargeek approved these changes Nov 12, 2025
View reviewed changes
Copy link
Contributor

@guitargeek guitargeek left a comment

Choose a reason for hiding this comment

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

LGTM when it works!

@vepadulano
Copy link
Member Author

vepadulano commented Jan 9, 2026

Missing things to address

  • Error in <DavixOpen>: can not open file "http://root.cern/files/tutorials/ntpl004_dimuon_v1.root" with davix: Result (Neon): Server certificate verification failed: issuer is not trusted after 3 attempts triggered by ntpl004_dimuon.C
  • fatal error: 'ROOT/RCanvas.hxx' file not found triggered by ntpl011_global_temperatures.C. This is understandable since RCanvas is not currently shipped by the wheels

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

Reviewers

@guitargeek guitargeek guitargeek approved these changes

@dpiparo dpiparo Awaiting requested review from dpiparo dpiparo is a code owner

Assignees

@vepadulano vepadulano

Labels

build-python-wheels Run the workflow to build Python wheels skip ci Skip the full builds on the actions runners skip code analysis Skip the code analysis CI steps for this PR, including verifying clang-formatting and running Ruff.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vepadulano @guitargeek