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

Use numpy build options for CPU optimizations #232

Closed

Conversation

AlexKurek
Copy link
Contributor

@AlexKurek AlexKurek commented Jan 23, 2025

Description

https://numpy.org/doc/stable/reference/simd/build-options.html#i-am-building-numpy-for-my-local-use
This disables CPU dispatching and selects the architecture.
Base branch: #222.

@tikk3r
Copy link
Owner

tikk3r commented Jan 23, 2025

If this depends on #222 then I think it should merge to that and not the main branch.

if [ $MARCH = 'x86-64' ] && [ $MTUNE = 'generic' ]; then
pip install . -Csetup-args=-Dblas-order=mkl,openblas,blis -Csetup-args=-Dlapack-order=mkl,openblas,lapack
else
pip install . -Csetup-args=-Dcpu-baseline="native" -Csetup-args=-Dcpu-dispatch="none" -Csetup-args=-Dblas-order=mkl,openblas,blis -Csetup-args=-Dlapack-order=mkl,openblas,lapack
Copy link
Owner

Choose a reason for hiding this comment

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

native cannot be used, as these recipes do not "build for local use" as that part of the numpy docs aims for.

@AlexKurek
Copy link
Contributor Author

If this depends on #222 then I think it should merge to that and not the main branch.

Im sorry, I was doing it first time, thought it will merge to #222 not to main.

@AlexKurek AlexKurek closed this Jan 23, 2025
@tikk3r
Copy link
Owner

tikk3r commented Jan 23, 2025

No worries, just trying to prevent multiple PRs from getting out of sync.

@AlexKurek AlexKurek deleted the Use-Numpy-build-options branch January 26, 2025 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants