roms-tools 1.6.1__py3-none-any.whl → 1.7.0__py3-none-any.whl

roms_tools/tests/test_setup/test_river_forcing.py

@@ -0,0 +1,366 @@
+from roms_tools import RiverForcing, Grid
+import xarray as xr
+import numpy as np
+from datetime import datetime
+import textwrap
+from pathlib import Path
+import pytest
+from conftest import calculate_file_hash
+from roms_tools.setup.download import download_river_data
+
+
+@pytest.fixture
+def river_forcing_climatology():
+    """Fixture for creating a RiverForcing object from the global Dai river dataset."""
+    grid = Grid(
+        nx=18, ny=18, size_x=800, size_y=800, center_lon=-18, center_lat=65, rot=20, N=3
+    )
+
+    start_time = datetime(1998, 1, 1)
+    end_time = datetime(1998, 3, 1)
+
+    return RiverForcing(
+        grid=grid,
+        start_time=start_time,
+        end_time=end_time,
+        convert_to_climatology="always",
+    )
+
+
+@pytest.fixture
+def river_forcing_for_grid_that_straddles_dateline():
+    """Fixture for creating a RiverForcing object from the global Dai river dataset for
+    a grid that straddles the dateline."""
+
+    grid = Grid(
+        nx=18,
+        ny=18,
+        size_x=1500,
+        size_y=1500,
+        center_lon=-0,
+        center_lat=65,
+        rot=20,
+        N=3,
+    )
+    start_time = datetime(1998, 1, 1)
+    end_time = datetime(1998, 3, 1)
+
+    return RiverForcing(
+        grid=grid,
+        start_time=start_time,
+        end_time=end_time,
+    )
+
+
+def compare_dictionaries(dict1, dict2):
+    assert dict1.keys() == dict2.keys()
+
+    for key in dict1.keys():
+        assert np.array_equal(dict1[key], dict2[key])
+
+
+@pytest.mark.parametrize(
+    "river_forcing_fixture",
+    ["river_forcing", "river_forcing_for_grid_that_straddles_dateline"],
+)
+def test_successful_initialization_with_climatological_dai_data(
+    river_forcing_fixture, request
+):
+
+    river_forcing = request.getfixturevalue(river_forcing_fixture)
+
+    assert isinstance(river_forcing.ds, xr.Dataset)
+    assert len(river_forcing.ds.nriver) > 0
+    assert len(river_forcing.original_indices["name"]) > 0
+    assert "river_volume" in river_forcing.ds
+    assert "river_tracer" in river_forcing.ds
+    assert river_forcing.climatology
+    assert "river_time" in river_forcing.ds
+    assert hasattr(
+        river_forcing.ds.river_time,
+        "cycle_length",
+    )
+    assert hasattr(river_forcing.ds, "climatology")
+
+
+def test_successful_initialization_with_monthly_dai_data(river_forcing_no_climatology):
+
+    assert isinstance(river_forcing_no_climatology.ds, xr.Dataset)
+    assert "river_volume" in river_forcing_no_climatology.ds
+    assert "river_tracer" in river_forcing_no_climatology.ds
+    assert "river_time" in river_forcing_no_climatology.ds
+    assert not river_forcing_no_climatology.climatology
+    assert not hasattr(
+        river_forcing_no_climatology.ds.river_time,
+        "cycle_length",
+    )
+    assert not hasattr(river_forcing_no_climatology.ds, "climatology")
+
+
+def test_reproducibility(river_forcing, river_forcing_climatology):
+
+    xr.testing.assert_allclose(river_forcing.ds, river_forcing_climatology.ds)
+
+    compare_dictionaries(
+        river_forcing.original_indices, river_forcing_climatology.original_indices
+    )
+    compare_dictionaries(
+        river_forcing.updated_indices, river_forcing_climatology.updated_indices
+    )
+
+
+def test_reproducibility_indices(river_forcing, river_forcing_no_climatology):
+
+    compare_dictionaries(
+        river_forcing.original_indices, river_forcing_no_climatology.original_indices
+    )
+    compare_dictionaries(
+        river_forcing.updated_indices, river_forcing_no_climatology.updated_indices
+    )
+
+
+@pytest.mark.parametrize(
+    "river_forcing_fixture",
+    ["river_forcing_climatology", "river_forcing_no_climatology"],
+)
+def test_constant_tracers(river_forcing_fixture, request):
+    river_forcing = request.getfixturevalue(river_forcing_fixture)
+
+    np.testing.assert_allclose(
+        river_forcing.ds.river_tracer.isel(ntracers=0).values, 17.0, atol=0
+    )
+    np.testing.assert_allclose(
+        river_forcing.ds.river_tracer.isel(ntracers=1).values, 1.0, atol=0
+    )
+
+
+@pytest.mark.parametrize(
+    "river_forcing_fixture",
+    ["river_forcing_climatology", "river_forcing_no_climatology"],
+)
+def test_river_locations_are_along_coast(river_forcing_fixture, request):
+    river_forcing = request.getfixturevalue(river_forcing_fixture)
+
+    mask = river_forcing.grid.ds.mask_rho
+    faces = (
+        mask.shift(eta_rho=1)
+        + mask.shift(eta_rho=-1)
+        + mask.shift(xi_rho=1)
+        + mask.shift(xi_rho=-1)
+    )
+    coast = (1 - mask) * (faces > 0)
+
+    indices = river_forcing.updated_indices
+    for i in range(len(indices["station"])):
+        eta_rho = indices["eta_rho"][i]
+        xi_rho = indices["xi_rho"][i]
+        assert coast[eta_rho, xi_rho]
+        assert river_forcing.grid.ds["river_flux"][eta_rho, xi_rho] > 0
+
+
+def test_missing_source_name():
+
+    grid = Grid(
+        nx=2,
+        ny=2,
+        size_x=500,
+        size_y=1000,
+        center_lon=0,
+        center_lat=55,
+        rot=10,
+        N=3,  # number of vertical levels
+    )
+
+    with pytest.raises(ValueError, match="`source` must include a 'name'."):
+        RiverForcing(
+            grid=grid,
+            start_time=datetime(1998, 1, 1),
+            end_time=datetime(1998, 3, 1),
+            source={"path": "river_data.nc"},
+        )
+
+
+def test_no_rivers_found():
+
+    # Create a grid over open ocean
+    grid = Grid(
+        nx=2, ny=2, size_x=50, size_y=50, center_lon=0, center_lat=55, rot=10, N=3
+    )
+    with pytest.raises(
+        ValueError,
+        match="No relevant rivers found.",
+    ):
+
+        RiverForcing(
+            grid=grid,
+            start_time=datetime(1998, 1, 1),
+            end_time=datetime(1998, 3, 1),
+        )
+
+
+def test_reproducibility_same_grid(river_forcing):
+
+    the_same_river_forcing = RiverForcing(
+        grid=river_forcing.grid,
+        start_time=datetime(1998, 1, 1),
+        end_time=datetime(1998, 3, 1),
+    )
+
+    assert river_forcing == the_same_river_forcing
+
+
+def test_update_river_flux_variable_without_conflicts(river_forcing, tmp_path):
+
+    fname = download_river_data("dai_trenberth_may2019.nc")
+    ds = xr.open_dataset(fname, decode_times=False)
+    # only keep the 300 biggest rivers, which will lower the total relevant river number
+    ds = ds.isel(station=slice(None, 300))
+    filepath = tmp_path / "test.nc"
+    ds.to_netcdf(filepath)
+
+    another_river_forcing = RiverForcing(
+        grid=river_forcing.grid,
+        start_time=datetime(1998, 1, 1),
+        end_time=datetime(1998, 3, 1),
+        source={"name": "DAI", "path": filepath},
+    )
+
+    assert isinstance(another_river_forcing.ds, xr.Dataset)
+
+
+def test_river_forcing_plot(river_forcing):
+    """Test plot method."""
+
+    river_forcing.plot_locations()
+    river_forcing.plot("river_volume")
+    river_forcing.plot("river_temperature")
+    river_forcing.plot("river_salinity")
+
+
+def test_river_forcing_save(river_forcing, tmp_path):
+    """Test save method."""
+
+    for file_str, grid_file_str in zip(
+        ["test_rivers", "test_rivers.nc"], ["test_grid", "test_grid.nc"]
+    ):
+        # Create a temporary filepath using the tmp_path fixture
+        for filepath, grid_filepath in zip(
+            [tmp_path / file_str, str(tmp_path / file_str)],
+            [tmp_path / grid_file_str, str(tmp_path / grid_file_str)],
+        ):  # test for Path object and str
+
+            # Test saving without partitioning
+            saved_filenames = river_forcing.save(filepath, grid_filepath)
+            # Check if the .nc file was created
+            filepath = Path(filepath).with_suffix(".nc")
+            grid_filepath = Path(grid_filepath).with_suffix(".nc")
+            assert saved_filenames == [filepath, grid_filepath]
+            assert filepath.exists()
+            assert grid_filepath.exists()
+            # Clean up the .nc file
+            filepath.unlink()
+            grid_filepath.unlink()
+
+            # Test saving with partitioning
+            saved_filenames = river_forcing.save(
+                filepath, grid_filepath, np_eta=3, np_xi=3
+            )
+
+            filepath_str = str(filepath.with_suffix(""))
+            grid_filepath_str = str(grid_filepath.with_suffix(""))
+            expected_filepath_list = [
+                Path(filepath_str + f".{index}.nc") for index in range(9)
+            ] + [Path(grid_filepath_str + f".{index}.nc") for index in range(9)]
+            assert saved_filenames == expected_filepath_list
+            for expected_filepath in expected_filepath_list:
+                assert expected_filepath.exists()
+                expected_filepath.unlink()
+
+
+def test_roundtrip_yaml(river_forcing, tmp_path):
+    """Test that creating an RiverForcing object, saving its parameters to yaml file,
+    and re-opening yaml file creates the same object."""
+
+    # Create a temporary filepath using the tmp_path fixture
+    file_str = "test_yaml"
+    for filepath in [
+        tmp_path / file_str,
+        str(tmp_path / file_str),
+    ]:  # test for Path object and str
+
+        river_forcing.to_yaml(filepath)
+
+        river_forcing_from_file = RiverForcing.from_yaml(filepath)
+
+        assert river_forcing == river_forcing_from_file
+
+        filepath = Path(filepath)
+        filepath.unlink()
+
+
+def test_files_have_same_hash(river_forcing, tmp_path):
+
+    yaml_filepath = tmp_path / "test_yaml.yaml"
+    filepath1 = tmp_path / "test1.nc"
+    filepath2 = tmp_path / "test2.nc"
+    grid_filepath1 = tmp_path / "grid_test1.nc"
+    grid_filepath2 = tmp_path / "grid_test2.nc"
+
+    river_forcing.to_yaml(yaml_filepath)
+    river_forcing.save(filepath1, grid_filepath1)
+    rf_from_file = RiverForcing.from_yaml(yaml_filepath)
+    rf_from_file.save(filepath2, grid_filepath2)
+
+    hash1 = calculate_file_hash(filepath1)
+    hash2 = calculate_file_hash(filepath2)
+
+    assert hash1 == hash2, f"Hashes do not match: {hash1} != {hash2}"
+
+    yaml_filepath.unlink()
+    filepath1.unlink()
+    filepath2.unlink()
+    grid_filepath1.unlink()
+    grid_filepath2.unlink()
+
+
+def test_from_yaml_missing_initial_conditions(tmp_path):
+    yaml_content = textwrap.dedent(
+        """\
+    ---
+    roms_tools_version: 0.0.0
+    ---
+    Grid:
+      nx: 100
+      ny: 100
+      size_x: 1800
+      size_y: 2400
+      center_lon: -10
+      center_lat: 61
+      rot: -20
+      topography_source: ETOPO5
+      hmin: 5.0
+    """
+    )
+
+    # Create a temporary filepath using the tmp_path fixture
+    file_str = "test_yaml"
+    for yaml_filepath in [
+        tmp_path / file_str,
+        str(tmp_path / file_str),
+    ]:  # test for Path object and str
+
+        # Write YAML content to file
+        if isinstance(yaml_filepath, Path):
+            yaml_filepath.write_text(yaml_content)
+        else:
+            with open(yaml_filepath, "w") as f:
+                f.write(yaml_content)
+
+        with pytest.raises(
+            ValueError,
+            match="No RiverForcing configuration found in the YAML file.",
+        ):
+            RiverForcing.from_yaml(yaml_filepath)
+
+        yaml_filepath = Path(yaml_filepath)
+        yaml_filepath.unlink()

roms_tools/tests/test_setup/test_surface_forcing.py

@@ -748,9 +748,7 @@ def test_from_yaml_missing_surface_forcing(tmp_path, use_dask):
       center_lat: 61
       rot: -20
       topography_source: ETOPO5
-      smooth_factor: 8
       hmin: 5.0
-      rmax: 0.2
     """
     )
 

roms_tools/tests/test_setup/test_tides.py

@@ -267,9 +267,7 @@ def test_from_yaml_missing_tidal_forcing(tmp_path, use_dask):
       center_lat: 61
       rot: -20
       topography_source: ETOPO5
-      smooth_factor: 8
       hmin: 5.0
-      rmax: 0.2
     """
     )
 

roms_tools/tests/test_setup/test_validation.py

@@ -23,6 +23,8 @@ def _get_fname(name):
         "bgc_surface_forcing_from_climatology",
         "boundary_forcing",
         "bgc_boundary_forcing_from_climatology",
+        "river_forcing",
+        "river_forcing_no_climatology",
     ],
 )
 # this test will not be run by default
@@ -60,6 +62,8 @@ def test_save_results(forcing_fixture, request):
         "bgc_surface_forcing_from_climatology",
         "boundary_forcing",
         "bgc_boundary_forcing_from_climatology",
+        "river_forcing",
+        "river_forcing_no_climatology",
     ],
 )
 def test_check_results(forcing_fixture, request):

roms_tools/utils.py

@@ -176,11 +176,12 @@ def partition(
         ]
 
     if np_eta > 1:
-        partitioned_sizes["eta_rho"] = (
-            [eta_rho_domain_size + n_eta_ghost_cells]
-            + [eta_rho_domain_size] * (np_eta - 2)
-            + [eta_rho_domain_size + n_eta_ghost_cells]
-        )
+        if "eta_rho" in dims_to_partition:
+            partitioned_sizes["eta_rho"] = (
+                [eta_rho_domain_size + n_eta_ghost_cells]
+                + [eta_rho_domain_size] * (np_eta - 2)
+                + [eta_rho_domain_size + n_eta_ghost_cells]
+            )
         if "eta_psi" in dims_to_partition:
             partitioned_sizes["eta_psi"] = (
                 [n_eta_ghost_cells + eta_psi_domain_size]
@@ -195,11 +196,12 @@ def partition(
             )
 
     if np_xi > 1:
-        partitioned_sizes["xi_rho"] = (
-            [xi_rho_domain_size + n_xi_ghost_cells]
-            + [xi_rho_domain_size] * (np_xi - 2)
-            + [xi_rho_domain_size + n_xi_ghost_cells]
-        )
+        if "xi_rho" in dims_to_partition:
+            partitioned_sizes["xi_rho"] = (
+                [xi_rho_domain_size + n_xi_ghost_cells]
+                + [xi_rho_domain_size] * (np_xi - 2)
+                + [xi_rho_domain_size + n_xi_ghost_cells]
+            )
         if "xi_psi" in dims_to_partition:
             partitioned_sizes["xi_psi"] = (
                 [n_xi_ghost_cells + xi_psi_domain_size]

{roms_tools-1.6.1.dist-info → roms_tools-1.7.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: roms-tools
-Version: 1.6.1
+Version: 1.7.0
 Summary: Tools for running and analysing UCLA-ROMS simulations
 Author-email: Nora Loose <nora.loose@gmail.com>, Thomas Nicholas <tom@cworthy.org>
 License: Apache-2
@@ -26,13 +26,13 @@ Requires-Dist: matplotlib
 Requires-Dist: cartopy
 Requires-Dist: packaging
 Requires-Dist: scipy
-Requires-Dist: gcm-filters
+Requires-Dist: gcm_filters
 Requires-Dist: netcdf4
 Requires-Dist: PyYAML
 Requires-Dist: pyamg
 Requires-Dist: bottleneck
 Provides-Extra: dask
-Requires-Dist: dask; extra == "dask"
+Requires-Dist: dask[diagnostics]; extra == "dask"
 
 # ROMS-Tools
 [![Conda version](https://img.shields.io/conda/vn/conda-forge/roms-tools.svg)](https://anaconda.org/conda-forge/roms-tools)
@@ -47,7 +47,7 @@ Requires-Dist: dask; extra == "dask"
 
 A suite of python tools for setting up a [ROMS](https://github.com/CESR-lab/ucla-roms) simulation.
 
-<mark>**DISCLAIMER**: This project is **ALPHA** and will be **EXPERIMENTAL** at least throughout Fall 2024. We do not recommend to use this software before this ALPHA development and test phase is completed.</mark>
+<mark>**DISCLAIMER**: This project is **ALPHA** and will be **EXPERIMENTAL** at least throughout 2024. We do not recommend to use this software before this ALPHA development and test phase is completed.</mark>
 
 ## Installation
 
@@ -98,7 +98,7 @@ If you want to use `ROMS-Tools` together with dask (which we recommend), you can
 install `ROMS-Tools` along with the additional dependency via:
 
 ```bash
-pip install -e .[dask]
+pip install -e ".[dask]"
 ```