search

cds-astro / cds-healpix-python

cdshealpix python package
https://cds-astro.github.io/cds-healpix-python/
BSD 3-Clause "New" or "Revised" License
6 stars 3 forks source link

Add conda build for ARM architectures #31

Open camposandro opened 1 month ago

camposandro commented 1 month ago

I noticed that conda is currently unable to resolve the latest hats v0.4.0 release in ARM-based architectures since no cds-healpix conda build exists for ARM Mac/Linux. From your previous releases on PyPI it seems like it's possible. Do you know if specifying those build platforms in conda-forge.yml would be enough to tackle this issue? Relates to https://github.com/cds-astro/cds-healpix-python/issues/11.

ManonMarchand commented 1 month ago

Yes we got a ci failure during the release (see https://github.com/cds-astro/cds-healpix-python/actions/runs/11384977564). One of our dependencies (pyo3) took more than 6h to build.

It might potentially be fixed when https://github.com/PyO3/maturin/issues/2256 is fixed.

ManonMarchand commented 1 month ago

Wait. You have an issue on MAC? This is unexpected, that part did not fail :frowning: Can you confirm?

camposandro commented 1 month ago

Yes, I'm testing on a Mac M2. Installing via pip works like a charm, while conda install conda-forge::cdshealpix is unable to resolve the package: PackagesNotFoundError: The following packages are not available from current channels: - conda-forge::cdshealpix.

fxpineau commented 1 month ago

No file corresponding to your needs here? Which MacOS and Python versions do you have? Oups ... with conda... thus here (not as easy to understand as pipy).

fxpineau commented 1 month ago

True: contrary to pipy, we have never built conda packages (neither cdshealpix nor mocpy) for Mac with ARM processors, check the list here. We have to check how to include osx-arm64 to our conda-forge recipes... Is it blocking for you (or not since you succeed with pypi)?

fxpineau commented 1 month ago

Meanwhile, a (unsatisfactory) solution is to create a x64 conda environment, see this stackoverflow question.

camposandro commented 1 month ago

It'd be great if we could have this on the short-term, but it isn't pressing. Thanks for having a look at it!

fxpineau commented 1 month ago

As a first step, this PR on conda-forge must be accepted (not in my hands). So, wait and see....