-
Notifications
You must be signed in to change notification settings - Fork 66
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
Updated CI #253
Updated CI #253
Conversation
cc @alecjacobson now it's working |
I can add it if you want.. but will be deprecated soon, I've seen https://devguide.python.org/versions/ |
Done, you can merge it if you want @alecjacobson |
Not exists python3.8 for arm64 macos |
Thanks. Before this pull request the jobs were separated into each .whl
rather than 5 jobs. Could you please switch it back to that? 5 jobs takes
much longer for the full CI to finish
…On Sat, Feb 15, 2025, 8:05 AM Johnny ***@***.***> wrote:
Not exists python3.8 for arm64 macos
—
Reply to this email directly, view it on GitHub
<#253 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AARDJGJDFZANZYZQTDCUJET2P43PZAVCNFSM6AAAAABXE4UOVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRQHEYTQNRTHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
[image: johnnynunez]*johnnynunez* left a comment
(libigl/libigl-python-bindings#253)
<#253 (comment)>
Not exists python3.8 for arm64 macos
—
Reply to this email directly, view it on GitHub
<#253 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AARDJGJDFZANZYZQTDCUJET2P43PZAVCNFSM6AAAAABXE4UOVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRQHEYTQNRTHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
1h42m vs 23m for reference |
Its windows... |
Is it possible to separate the "jobs" for each ".whl"? Like before. This
parallelizes over the jobs and is much faster overall (even if each windows
.whl is slower)
…On Sat, Feb 15, 2025, 7:22 PM Johnny ***@***.***> wrote:
1h42m vs 23m for reference
Its windows...
—
Reply to this email directly, view it on GitHub
<#253 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AARDJGNKR5MARLYIYRPSDE32P7K4FAVCNFSM6AAAAABXE4UOVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRRGE3DAOBXGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
[image: johnnynunez]*johnnynunez* left a comment
(libigl/libigl-python-bindings#253)
<#253 (comment)>
1h42m vs 23m for reference
Its windows...
—
Reply to this email directly, view it on GitHub
<#253 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AARDJGNKR5MARLYIYRPSDE32P7K4FAVCNFSM6AAAAABXE4UOVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMNRRGE3DAOBXGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
yeah for sure! |
|
@alecjacobson for me all is done! Thank you for the repo and enjoy it :) |
@alecjacobson will you merge it? And when will be new release? |
This pull request includes several updates to the CI/CD workflows and configuration files to enhance compatibility and streamline the build process. The most important changes include adding a new platform to the build matrix, refining the build and test steps, and updating the
pyproject.toml
file to specify architecture and image details.CI/CD Workflow Updates:
.github/workflows/pip.yml
: Addedubuntu-24.04-arm
to the platform matrix to expand the build coverage..github/workflows/wheels.yml
:build_sdist
andbuild_wheels
jobs, ensuring failure ontwine check
errors and specifying architecture and Python builds at the top level for clarity.release_artifacts
job to handle artifact release on tag pushes, including downloading artifacts, listing them, and usingrelease-action
andgh-action-pypi-publish
for publishing.Configuration File Updates:
pyproject.toml
:archs = "auto64"
to thecibuildwheel
tool configuration to automatically detect the architecture.manylinux
images forx86_64
andaarch64
to ensure consistent build environments.*Include Windows Arch build wheel.