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

Fix Windows wheels #693

Merged
merged 50 commits into from
Apr 24, 2024
Merged

Fix Windows wheels #693

merged 50 commits into from
Apr 24, 2024

Conversation

AmintorDusko
Copy link
Contributor

@AmintorDusko AmintorDusko commented Apr 22, 2024

Context: We add tests for Windows Wheels, fix ill-defined caching, and set the proper backend for LKokkos wheels.

Description of the Change: Move tests to happen after patching the wheels, set a proper key for cache (compatible with Windows system), and set environment variables to set the proper backend to be built.
Building the LKokkos wheels is out-of-scope for this PR, and will be revisited when sensible.

Benefits: Proper wheels tested and working as intended.

Possible Drawbacks: For now we will not be building wheels for LKokkos on Windows.

Related GitHub Issues:

@AmintorDusko AmintorDusko added the ci:build_wheels Activate wheel building. label Apr 22, 2024
@AmintorDusko
Copy link
Contributor Author

[sc-59304]

@AmintorDusko AmintorDusko marked this pull request as ready for review April 22, 2024 20:23
@AmintorDusko AmintorDusko marked this pull request as draft April 23, 2024 12:19
@AmintorDusko AmintorDusko marked this pull request as ready for review April 23, 2024 12:40
@AmintorDusko AmintorDusko requested review from vincentmr and a team April 23, 2024 12:41
Copy link
Contributor

@vincentmr vincentmr 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 to me, thanks @AmintorDusko .

Copy link
Member

@maliasadi maliasadi left a comment

Choose a reason for hiding this comment

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

Thanks @AmintorDusko 🙌

.github/workflows/wheel_win_x86_64.yml Outdated Show resolved Hide resolved
.github/workflows/wheel_win_x86_64.yml Outdated Show resolved Hide resolved
.github/workflows/wheel_win_x86_64.yml Outdated Show resolved Hide resolved
@AmintorDusko AmintorDusko merged commit 43f7a91 into master Apr 24, 2024
81 checks passed
@AmintorDusko AmintorDusko deleted the fix_Windows_wheels branch April 24, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:build_wheels Activate wheel building.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants