Skip to content

Add CI to release nightly free-threaded wheels#7481

Merged
stefanv merged 5 commits intoscikit-image:mainfrom
andfoy:add_free_threaded_wheel_ci
Aug 13, 2024
Merged

Add CI to release nightly free-threaded wheels#7481
stefanv merged 5 commits intoscikit-image:mainfrom
andfoy:add_free_threaded_wheel_ci

Conversation

@andfoy
Copy link
Copy Markdown
Contributor

@andfoy andfoy commented Aug 1, 2024

Description

See #7464

This PR serves as a followup to #7463, adding a separate workflow to produce, test and upload nightly wheels to the Scientific Python Anaconda channel.

Checklist

Release note

For maintainers and optionally contributors, please refer to the instructions on how to document this PR for the release notes.

...

@andfoy andfoy changed the title Add CI to release nighlty free-threaded wheels Add CI to release nightly free-threaded wheels Aug 1, 2024
@andfoy
Copy link
Copy Markdown
Contributor Author

andfoy commented Aug 2, 2024

I'm currently investigating an error in the test suite that is occurring under the latest 3.13 beta

@andfoy andfoy force-pushed the add_free_threaded_wheel_ci branch from 3d49c78 to 67b8197 Compare August 7, 2024 16:01
@andfoy
Copy link
Copy Markdown
Contributor Author

andfoy commented Aug 7, 2024

Wheels are being built successfully in my fork: https://github.com/andfoy/scikit-image/actions/runs/10287231013/job/28469781344

@andfoy andfoy marked this pull request as ready for review August 7, 2024 16:03
[[1, 2, 3, 1, 2], [3, 3, 5, 4, 2], [3, 4, 5, 4, 2], [3, 3, 2, 1, 2]]
)
out = np.empty_like(image)
out = image.copy()
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This seems to be a legit issue in this function. out needs to be initialized as a copy of image, for some reason empty_like returned a previous chunk of memory which had similar values to image, thus it was passing before the memory allocator changes in CPython 3.13

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.

Uff, that's a good catch. Good reminder to never rely on the content of np.empty.

@lagru lagru added the 🤖 type: Infrastructure CI, packaging, tools and automation label Aug 7, 2024
Copy link
Copy Markdown
Member

@lagru lagru left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks @andfoy! 😊

What do you think about merging build_linux_39_and_above_wheels and build_linux_free_threaded_wheels? Would that be possible to make the extra CIBW_FREE_THREADED_SUPPORT and the like conditional on cp313t-* in the matrix? Maybe at a later point when we can remove the nightly installation?

@andfoy
Copy link
Copy Markdown
Contributor Author

andfoy commented Aug 12, 2024

@lagru, thanks for the review, changes should be now up in 1d7c879, which is working in my fork: https://github.com/andfoy/scikit-image/actions/runs/10354531538/job/28660020992

@lagru lagru added the 👍 1st approval Needs 2nd approval to merge label Aug 13, 2024
@stefanv stefanv merged commit 48c7daa into scikit-image:main Aug 13, 2024
@stefanv
Copy link
Copy Markdown
Member

stefanv commented Aug 13, 2024

Thank you @andfoy

@stefanv stefanv added this to the 0.25 milestone Aug 13, 2024
@andfoy andfoy deleted the add_free_threaded_wheel_ci branch August 14, 2024 05:00
echo "CIBW_FREE_THREADED_SUPPORT=1" >> "$GITHUB_ENV"
echo "CIBW_BEFORE_BUILD=$NIGHTLY_INSTALL" >> "$GITHUB_ENV"
echo "CIBW_BEFORE_TEST=$NIGHTLY_INSTALL" >> "$GITHUB_ENV"
echo "CIBW_TEST_COMMAND=$TEST_COMMAND" >> "$GITHUB_ENV"
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.

Wouldn't have known how to do this! Thanks for figuring it out @andfoy. 😊

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

Labels

🤖 type: Infrastructure CI, packaging, tools and automation 👍 1st approval Needs 2nd approval to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants