Fix pandas FutureWarnings in test cases, update CI deps (dynamicslab#207

)

Update CI dependencies, fix FutureWarnings about datetime comparisons.

.github/workflows/docs-ci.yml

@@ -9,8 +9,8 @@ jobs:
   docs:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v2
-    - uses: actions/setup-python@v2
+    - uses: actions/checkout@v3
+    - uses: actions/setup-python@v4
       with:
         python-version: '3.8'
     - name: Install pandoc

.github/workflows/package-install.yml

@@ -10,9 +10,9 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
 
-      - uses: actions/setup-python@v2
+      - uses: actions/setup-python@v4
         with:
           python-version: '3.8'
 

.github/workflows/publish-pypi.yml

@@ -10,7 +10,7 @@ jobs:
     steps:
     - uses: actions/checkout@master
     - name: Set up Python 3.8
-      uses: actions/setup-python@v2
+      uses: actions/setup-python@v4
       with:
         python-version: '3.8'
     - name: Install pypa/build

.github/workflows/pytest-ci.yml

@@ -16,7 +16,7 @@ jobs:
 
     steps:
       # Checkout the repository under $GITHUB_WORKSPACE
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
 
       # initialize conda
       - name: Conda setup
@@ -26,7 +26,7 @@ jobs:
           python-version: '3.10'
 
       # cache the conda installation to speedup the CI runs
-      - uses: actions/cache@v2
+      - uses: actions/cache@v3
         id: cache
         with:
           path: /usr/share/miniconda/envs/neuralhydrology

test/test_config_runs.py

@@ -1,6 +1,5 @@
 """Integration tests that perform full runs. """
 import pickle
-from datetime import datetime
 from pathlib import Path
 from typing import Dict, Tuple, Callable
 
@@ -161,7 +160,7 @@ def test_multi_timescale_regression(get_config: Fixture[Callable[[str], dict]],
     daily_results = results['1D']['xr']
     assert (results['1D']['xr']['time_step'].values == [0]).all()
     assert pd.to_datetime(daily_results['date'].values[0]) == test_start_date
-    assert pd.to_datetime(daily_results['date'].values[-1]) == test_end_date.date()
+    assert pd.to_datetime(daily_results['date'].values[-1]) == test_end_date.floor('D')
     assert len(daily_results['qobs_mm_per_hour_obs']) == len(discharge) // 24
 
     assert len(discharge) == len(hourly_results)
@@ -213,8 +212,8 @@ def _check_results(config: Config, basin: str, discharge: pd.Series = None):
 
     results = _get_basin_results(config.run_dir, 1)[basin]['1D']['xr'].isel(time_step=-1)
 
-    assert pd.to_datetime(results['date'].values[0]) == test_start_date.date()
-    assert pd.to_datetime(results['date'].values[-1]) == test_end_date.date()
+    assert pd.to_datetime(results['date'].values[0]) == test_start_date.floor('D')
+    assert pd.to_datetime(results['date'].values[-1]) == test_end_date.floor('D')
 
     if discharge is None:
         discharge = _get_discharge(config, basin)
@@ -226,7 +225,7 @@ def _check_results(config: Config, basin: str, discharge: pd.Series = None):
     assert not pd.isna(results[f'{config.target_variables[0]}_sim']).any()
 
 
-def _get_test_start_end_dates(config: Config) -> Tuple[datetime, datetime]:
+def _get_test_start_end_dates(config: Config) -> Tuple[pd.Timestamp, pd.Timestamp]:
     test_start_date = pd.to_datetime(config.test_start_date, format='%d/%m/%Y')
     test_end_date = pd.to_datetime(config.test_end_date, format='%d/%m/%Y') + pd.Timedelta(days=1, seconds=-1)