Skip to content

Change default env behavior #33

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.

Sign up for GitHub

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

Merged
merged 4 commits into from
Jul 14, 2025
Merged

Change default env behavior #33

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a1542d2
skip healpix install by default
elynnwu Jul 14, 2025
ca5ff66
cleaner install
elynnwu Jul 14, 2025
4d1fb93
remove for dockerfile
elynnwu Jul 14, 2025
e0ba23b
skip healpix coord test if earth2grid not installed
brianhenn Jul 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ create_environment:
conda create -n $(ENVIRONMENT_NAME) python=3.11 $(CONDA_PACKAGES)
conda run --no-capture-output -n $(ENVIRONMENT_NAME) python -m pip install uv
conda run --no-capture-output -n $(ENVIRONMENT_NAME) uv pip install -c constraints.txt -e .[dev,docs]
conda run --no-capture-output -n $(ENVIRONMENT_NAME) uv pip install --no-build-isolation -c constraints.txt -e .[dev,docs,healpix]
conda run --no-capture-output -n $(ENVIRONMENT_NAME) uv pip install -r analysis-deps.txt

test:
Expand Down
2 changes: 0 additions & 2 deletions docker/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,6 @@ ENV FORCE_CUDA_EXTENSION=1
# install python deps
COPY requirements.txt /tmp/requirements.txt
RUN python3 -m pip install -r /tmp/requirements.txt
COPY requirements-healpix.txt /tmp/requirements-healpix.txt
RUN python3 -m pip install --no-build-isolation -r /tmp/requirements-healpix.txt

FROM base AS production

Expand Down
2 changes: 2 additions & 0 deletions fme/core/test_coordinates.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
HEALPixCoordinates,
HybridSigmaPressureCoordinate,
LatLonCoordinates,
e2ghpx,
)
from fme.core.mask_provider import MaskProvider

Expand Down Expand Up @@ -239,6 +240,7 @@ def test_healpix_ops_raises_value_error_with_mask():
healpix_coords.get_gridded_operations(mask_provider=mask_provider)


@pytest.mark.skipif(e2ghpx is None, reason="earth2grid healpix not available")
@pytest.mark.parametrize("pad", [True, False])
def test_healpix_coordinates_xyz(pad: bool, very_fast_only: bool):
if very_fast_only:
Expand Down