-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
CI ARM #7196
base: main
Are you sure you want to change the base?
CI ARM #7196
Conversation
Thanks for submitting this pull request! The maintainers of this repository would appreciate if you could update the CHANGELOG.md based on your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @johnnynunez, this is really useful! Please see my comments below.
Hi @johnnynunez please take a look at the ARM64 CI errors. You can ignore the Windows CI errors, they are unrelated.
|
This pull request includes significant updates to the
.github/workflows/ubuntu-openblas.yml
file, focusing on updating the Docker build and test processes, as well as removing unnecessary steps related to the ARM64 job.Key changes include:
Docker Build and Test Updates:
openblas-amd64-py312-dev
instead ofopenblas-amd64-py310-dev
.ARM64 Job Simplification:
skip-arm64-check-on-fork
job and its associated steps, simplifying the workflow.runs-on
parameter for theopenblas-arm64
job to useubuntu-24.04-arm
instead ofubuntu-latest
.