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

Added ARM64 support; Added build config with environment variables #116

Merged
merged 6 commits into from
May 27, 2021

Conversation

t20100
Copy link
Member

@t20100 t20100 commented May 27, 2021

This PR should add support for M1 by disabling SSE2 and AVX2 on arm64 machines.
It also adds env var to configure default build options (useful with cibuildwheel)

This is UNTESTED!

@t20100 t20100 added this to the 3.0.0 milestone May 27, 2021
@Anthchirp
Copy link
Contributor

I don't have access to an M1 either - maybe @ndevenish may be able to help out?

@ndevenish
Copy link

ndevenish commented May 27, 2021

FWIW pip install . and then python3 test/test.py works fine on 20ef138 (with a conda environment for h5py and numpy, native building numpy fails from sources). (Edit: On an M1 macbook, in case that wasn't clear from context)

@t20100
Copy link
Member Author

t20100 commented May 27, 2021

Great! Thanks for the testing!

cibuildwheel also manages to build on GHA with cfa0103 but no testing.

@t20100 t20100 requested review from kif and vasole May 27, 2021 09:05
@t20100 t20100 marked this pull request as ready for review May 27, 2021 09:05
@t20100
Copy link
Member Author

t20100 commented May 27, 2021

Ready for review.
It's been tested natively on M1 and I added config through env var. to enable arm64-macos wheel build on Github Actions through cibuildwheel

@t20100 t20100 changed the title Added support of ARM64 CPUs for Apple Silicon M1 support Added support of ARM64; Added build config with environment variables May 27, 2021
@t20100 t20100 changed the title Added support of ARM64; Added build config with environment variables Added ARM64 support; Added build config with environment variables May 27, 2021
@vasole vasole merged commit f1768e0 into main May 27, 2021
@vasole vasole deleted the arm64 branch May 27, 2021 14:46
@t20100
Copy link
Member Author

t20100 commented Jun 8, 2021

FYI, this has been released in v3.0.0 and pushed to pypi: https://pypi.org/project/hdf5plugin/

t20100 added a commit to t20100/hdf5plugin that referenced this pull request Sep 4, 2023
092190c38 Fix spelling errors (silx-kit#128)
b241ca0d0 Fix the CMake multiconfig var and test and tools handling (silx-kit#127)
7b33e4f19 Fix some make output issues (silx-kit#126)
51d221848 Documentation fixes and fixes from testing 1.1.1 (silx-kit#125)
4ae30429c CMake: (fix) HDF5 1.14.0 (silx-kit#121)
e1072ec5b Update Github actions  (silx-kit#118)
b021532ea Fix leak and consistify generic API with ZFP C API (silx-kit#117)
06408bb22 Enable CMake Testing (silx-kit#116)
eb5440043 Fix version info in docs
ad61dfa41 Revert "rework feat-cmake_tests CMake and replace scripts (silx-kit#111)" (silx-kit#115)
03b86413f rework feat-cmake_tests CMake and replace scripts (silx-kit#111)
08116fa9e Update README.md (silx-kit#114)
a10bd7bdb Merge branch 'master' of github.com:LLNL/H5Z-ZFP
1660d94a4 Fix h5repack test (silx-kit#107)
fd01bf1d7 mention generic interface
8d5ec2e74 Update docs (silx-kit#106)
c76f847d9 Added new endianness test (silx-kit#102)
8acf8244b fix reading of byte-swapped input files (silx-kit#95) (silx-kit#101)
62ab4eb6d Minor code clean-up. (silx-kit#103)
60032f640 Bugfix/hdf5 1.14.0 (silx-kit#99)
8fad85d72 CMake: (fix) MPI parallel HDF5 1.14.0 (silx-kit#97)
e81aec114 Don't flag a skipped test as a failed test.  (silx-kit#94)

git-subtree-dir: src/H5Z-ZFP
git-subtree-split: 092190c38d5760b3212ce4d96cced767b90aa189
t20100 added a commit to t20100/hdf5plugin that referenced this pull request Jul 4, 2024
f466775 Merge pull request silx-kit#116 from ayzk/master1
bde9259 fix a bug when dimension is corrected in compression but not in decompression
a92658e Merge pull request silx-kit#109 from vasole/master
a5d6039 Restrict the scope only to windows without MINGW as already done.
d0c7f6c Force static build of external libraries with MSVC
73065ab Remove gcc specific options

git-subtree-dir: src/SZ
git-subtree-split: f4667759ead6a902110e80ff838ccdfddbc8dcd7
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.

4 participants