cleaning (dependencies, remove xclim53 hacks)

.pre-commit-config.yaml

@@ -68,7 +68,7 @@ repos:
     rev: v1.8.0
     hooks:
       - id: numpydoc-validation
-        exclude: ^docs/|^tests/|src/xsdba/xclim_submodules/
+        exclude: ^docs/|^tests/
   - repo: https://github.com/python-jsonschema/check-jsonschema
     rev: 0.29.2
     hooks:

.readthedocs.yml

@@ -15,10 +15,6 @@ build:
   tools:
     python: "mambaforge-22.9"
   jobs:
-    pre_install:
-      # FIXME: This is a workaround to install xclim v0.53, which is not released.
-      - mamba install -y -n base -c conda-forge "xclim=0.52.2"
-      - python -m pip install git+https://github.com/Ouranosinc/xclim.git@main
     pre_build:
       - sphinx-apidoc -o docs/apidoc --private --module-first src/xsdba
       - sphinx-build -M gettext docs docs/_build

CHANGELOG.rst

@@ -10,7 +10,7 @@ Contributors: Éric Dupuis (:user:`coxipi`), Trevor James Smith (:user:`Zeitsper
 Changes
 ^^^^^^^
 * Split `sdba` from `xclim` and duplicate code where needed. (:pull:`8`)
-* `calendar` and `units` are  copy (or almost) of given modules in `xclim`. Perhaps in the future some functionalities can be put in a common generic module (:pull:`8`)
+* `units` are a copy (or almost) of given modules in `xclim`. A lot of duplicated code from xclim's `calendar` is also in xsdba's `base`. (:pull:`8`)
 
 
 .. _changes_0.1.0:

environment-dev.yml

@@ -17,6 +17,7 @@ dependencies:
   - scipy >=1.9.0
   - statsmodels
   - xarray >=2023.11.0
+  - xclim >= 0.53
   - yamale
   # Dev tools and testing
   - netcdf4

pyproject.toml

@@ -1,5 +1,3 @@
-# SPLIT: many checks removed
-
 [build-system]
 requires = ["flit_core >=3.9,<4"]
 build-backend = "flit_core.buildapi"
@@ -76,7 +74,8 @@ dev = [
   "ruff >=0.5.7",
   "pooch >=1.8.0",
   "pre-commit >=3.5.0",
-  "xdoctest>=1.1.5"
+  "xdoctest>=1.1.5",
+  "xclim >= 0.53"
 ]
 docs = [
   # Documentation and examples
@@ -98,7 +97,6 @@ docs = [
   "sphinxcontrib-bibtex",
   "sphinxcontrib-svg2pdfconverter[Cairosvg]"
 ]
-extras = ["xclim>=0.52"]
 all = ["xsdba[dev]", "xsdba[docs]"]
 
 [project.scripts]
@@ -257,6 +255,7 @@ allow_untyped_defs = true
 disable_error_code = "attr-defined"
 ignore_missing_imports = true
 
+# SPLIT: many checks removed
 [tool.numpydoc_validation]
 checks = [
   "all", # report on all checks, except the below

tox.ini

@@ -59,8 +59,6 @@ commands_pre =
     pip list
     pip check
 commands =
-    ; Install the development version of xclim until v0.53.0 is released
-    pip install git+https://github.com/Ouranosinc/xclim.git@main
     pytest --cov xsdba -m "not requires_atmosds" {posargs}
     ; Coveralls requires access to a repo token set in .coveralls.yml in order to report stats
     coveralls: - coveralls