Skip to content

Commit

Permalink
Constrain dask !=2024.12.0,!=2024.12.1 for performance (#926)
Browse files Browse the repository at this point in the history
* Remove `defaults` channel from conda env files
  • Loading branch information
tomvothecoder authored Feb 3, 2025
1 parent 060308b commit f28c1cc
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 11 deletions.
3 changes: 1 addition & 2 deletions conda-env/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
name: e3sm_diags_ci
channels:
- conda-forge
- defaults
dependencies:
# Base
# =================
Expand All @@ -15,7 +14,7 @@ dependencies:
- cartopy >=0.17.0
- cartopy_offlinedata
- cf-units
- dask <2024.12.0
- dask !=2024.12.0,!=2024.12.1
- lxml
- mache >=0.15.0
- matplotlib-base >=3.8.2
Expand Down
3 changes: 1 addition & 2 deletions conda-env/dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
name: e3sm_diags_dev
channels:
- conda-forge
- defaults
dependencies:
# Base
# =======================
Expand All @@ -13,7 +12,7 @@ dependencies:
- cartopy >=0.17.0
- cartopy_offlinedata
- cf-units
- dask <2024.12.0
- dask !=2024.12.0,!=2024.12.1
- lxml
- mache >=0.15.0
- matplotlib-base >=3.8.2
Expand Down
9 changes: 2 additions & 7 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ dependencies = [
# This package is not available on PyPI.
# "cartopy_offlinedata",
"cf-units",
"dask <2024.12.0",
"dask !=2024.12.0,!=2024.12.1",
"lxml",
"mache >=0.15.0",
"matplotlib >=3.8.2",
Expand All @@ -50,12 +50,7 @@ dependencies = [
[project.optional-dependencies]
test = ["pytest", "pytest-cov"]
docs = ["sphinx", "sphinx_rtd_theme", "sphinx-multiversion"]
dev = [
"pre-commit",
"types-PyYAML",
"ruff",
"mypy",
]
dev = ["pre-commit", "types-PyYAML", "ruff", "mypy"]

[project.urls]
Documentation = "https://docs.e3sm.org/e3sm_diags/_build/html/main/index.html"
Expand Down

0 comments on commit f28c1cc

Please sign in to comment.