Merge pull request #189 from phobson/uv-in-ci

use uv in CI

.github/workflows/check-test-coverage.yml

@@ -17,8 +17,11 @@ jobs:
     - name: Generate coverage report
       run: |
         python -m pip install --upgrade pip
-        pip install flake8 pytest
-        if [ -f requirements_dev.txt ]; then pip install -r requirements_dev.txt; fi
+        python -m pip install uv
+        uv venv
+        source .venv/bin/activate
+        uv pip install pytest flake8
+        if [ -f requirements_dev.txt ]; then uv pip install -r requirements_dev.txt; fi
         coverage run --source wqio check_wqio.py --doctest-modules --cov --cov-report=xml
     - name: Upload coverage reports to Codecov
       run: |

.github/workflows/python-runtests-basic.yml

@@ -26,8 +26,12 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install pytest
-        if [ -f requirements_dev.txt ]; then pip install -r requirements_dev.txt; fi
+        python -m pip install uv
+        uv venv
+        source .venv/bin/activate
+        uv pip install pytest
+        if [ -f requirements_dev.txt ]; then uv pip install -r requirements_dev.txt; fi
     - name: Test with pytest
       run: |
+        source .venv/bin/activate
         python check_wqio.py --doctest-modules

.github/workflows/python-runtests-img-comp.yml

@@ -22,9 +22,13 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install flake8 pytest
-        if [ -f requirements_dev.txt ]; then pip install -r requirements_dev.txt; fi
+        python -m pip install uv
+        uv venv
+        source .venv/bin/activate
+        uv pip install pytest
+        if [ -f requirements_dev.txt ]; then uv pip install -r requirements_dev.txt; fi
     - name: Test with pytest
       run: |
+        source .venv/bin/activate
         export MPL_IMGCOMP_TOLERANCE=20
         python check_wqio.py --strict --verbose

wqio/hydro.py

@@ -266,7 +266,7 @@ def __init__(
             },
             self.precipcol: {
                 "name": "Precip (mm)",
-                "ylabel": "%s Precip.\nDepth (mm)" % self.hydrofreq_label,
+                "ylabel": f"{self.hydrofreq_label} Precip.\nDepth (mm)",
                 "color": "DarkGreen",
                 "linewidth": 1.5,
                 "alpha": 0.4,

wqio/tests/helpers.py

@@ -219,8 +219,8 @@ def _show_system_info():  # pragma: no cover
     import pytest
 
     pyversion = sys.version.replace("\n", "")
-    print("Python version %s" % pyversion)
-    print("pytest version %d.%d.%d" % pytest.__versioninfo__)
+    print(f"Python version {pyversion}")
+    print(f"pytest version {pytest.__versioninfo__}")
 
     import numpy
 

wqio/tests/test_datacollections.py

@@ -507,8 +507,7 @@ def test_wilcoxon(dc):
         F,Outflow,62.0,,28.0,0.492459,,0.656642
         F,Reference,22.0,28.0,,0.952765,0.656642,
     """
-    with pytest.warns(UserWarning):
-        check_stat(known_csv, dc.wilcoxon(), comp=True)
+    check_stat(known_csv, dc.wilcoxon(), comp=True)
 
 
 @helpers.seed

wqio/utils/numutils.py

@@ -153,7 +153,7 @@ def process_p_vals(pval):
     elif pval > 1 or pval < 0:
         raise ValueError(f"p-values must be between 0 and 1 (not {pval})")
     else:
-        out = "%0.3f" % pval
+        out = f"{pval:0.3f}"
 
     return out
 
@@ -377,7 +377,7 @@ def pH_to_concentration(pH, *args):
 
     # check that we recieved a valid input:
     if pH < 0 or pH > 14:
-        raise ValueError("pH = %f but must be between 0 and 14" % pH)
+        raise ValueError(f"pH = {pH} but must be between 0 and 14")
 
     # avogadro's number (items/mole)
     avogadro = 6.0221413e23