Skip to content

Conversation

@siliataider
Copy link
Contributor

@siliataider siliataider commented Aug 11, 2025

This Pull request:

This PR integrates automated testing of Python and C++ tutorials into the Python wheels workflow: after building each Python wheel for multiple Python versions, the workflow now installs the corresponding wheel on the ubuntu runner and runs the list of tutorials.

Note

Only a selected subset of tutorials is currently tested, covering the following directories:

  • analysis/dataframe
  • analysis/tree
  • hist
  • io/ntuple
  • roofit/roofit

@siliataider siliataider added the build-python-wheels Run the workflow to build Python wheels label Aug 11, 2025
@github-actions
Copy link

github-actions bot commented Aug 11, 2025

Test Results

    19 files      19 suites   3d 10h 53m 49s ⏱️
 3 738 tests  3 738 ✅ 0 💤 0 ❌
69 326 runs  69 326 ✅ 0 💤 0 ❌

Results for commit 6499a87.

♻️ This comment has been updated with latest results.

@siliataider siliataider added build-python-wheels Run the workflow to build Python wheels and removed build-python-wheels Run the workflow to build Python wheels labels Aug 11, 2025
@siliataider siliataider reopened this Aug 11, 2025
@siliataider siliataider force-pushed the wheels-pytest branch 3 times, most recently from 34c7886 to 6373b89 Compare September 3, 2025 12:50
@hahnjo
Copy link
Member

hahnjo commented Sep 3, 2025

hm, you may want to implement the equivalent of 771bf406a149. There were three jobs running for old versions of this PR...

@siliataider
Copy link
Contributor Author

hm, you may want to implement the equivalent of 771bf406a149. There were three jobs running for old versions of this PR...

Thanks! #19818

@siliataider siliataider force-pushed the wheels-pytest branch 3 times, most recently from 1a1e48f to 903bca5 Compare October 7, 2025 10:05
@siliataider siliataider force-pushed the wheels-pytest branch 2 times, most recently from fced9a5 to c34b3ed Compare October 7, 2025 14:44
@siliataider siliataider force-pushed the wheels-pytest branch 2 times, most recently from 1828f8d to 6d1472e Compare October 28, 2025 11:45
@siliataider siliataider marked this pull request as ready for review October 28, 2025 14:17
@siliataider siliataider requested a review from dpiparo as a code owner October 28, 2025 14:17
@siliataider siliataider changed the title [WIP] Integrate testing to the Python wheels workflow [ci][Python] Integrate testing to the Python wheels workflow Oct 28, 2025
@siliataider siliataider force-pushed the wheels-pytest branch 3 times, most recently from f0a4051 to 849c79c Compare October 29, 2025 12:14
@siliataider siliataider force-pushed the wheels-pytest branch 5 times, most recently from 46eca93 to 8bef2eb Compare October 29, 2025 15:47
Copy link
Member

@vepadulano vepadulano left a comment

Choose a reason for hiding this comment

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

Great work! Having continuous testing enabled on the Python wheels will help towards a smooth deployment of the ROOT package on PyPI! I have left a couple of minor comments to be addressed before merging.

@siliataider siliataider force-pushed the wheels-pytest branch 2 times, most recently from 6ed779f to 61e6566 Compare October 30, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-python-wheels Run the workflow to build Python wheels in:CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants