Skip to content

Conversation

@elynnwu
Copy link
Contributor

@elynnwu elynnwu commented Jul 14, 2025

Skip installing healpix since it may not work for all platforms. This is currently not a publicly supported option, so we should not install by default.

@brianhenn
Copy link
Contributor

brianhenn commented Jul 14, 2025

For context, the reason for this is that earth2grid installation is finicky and appears to fail on Perlmutter and possibly other platforms, so we don't want the public repo's conda/docker builds to try to install it. That means that HealPIX training/inference presumably won't work in the public code, but that seems fine for now.

Copy link

@Arcomano1234 Arcomano1234 left a comment

Choose a reason for hiding this comment

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

LGTM!

@elynnwu elynnwu merged commit 1382d5f into main Jul 14, 2025
5 checks passed
brianhenn added a commit that referenced this pull request Jul 16, 2025
* skip healpix install by default

* cleaner install

* remove for dockerfile

* skip healpix coord test if earth2grid not installed

---------

Co-authored-by: Brian Henn <[email protected]>
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