Tags: xarray-contrib/cf-xarray
Tags
Raise ValueError in `_get_ordered_vertices()` with "mixed" order (#595) * Raise a ValueError in `_get_ordered_vertices()` with `"mixed"` core dim order * Replace ValueError with NotImplementedError * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Pascal Bourgault <bourgault.pascal@ouranos.ca>
Update `bounds_to_vertices` to handle descending arrays (#579) * Update `bounds_to_vertices` to handle descending arrays Add `_get_ordered_vertices()` to handle extraction of unique vertex values in ascending order * Update docstring * Update `_get_ordered_vertices()` to preserve order of values * Fix `_get_ordered_vertices()` to preserve leading dims - Also handle strictly monotonic bounds * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Fix casting issue with numpy 1.26.4 * Update docstrings for clarity * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Fix logic for checking strictly monotonic * Fix logic for checking strictly monotonic * Delete qa/mvce.py * Update `bounds_to_vertices to sort on coordinates - Add `_get_core_dim_orders()` to get core dim orders for sorting vertices - Update `_bounds_helper()` to accept `core_dim_orders` param - Update `_get_ordered_vertices()` to sort on `core_dim_orders` param - Fix `_get_ordered_vertices()` to get correct array values for vertices based on order of core dims * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Fix 3d vertices test * Add typecasting to prevent numpy 1.26.4 issue * Add typecasting to prevent numpy 1.26.4 issue * Add typecasting to prevent numpy 1.26.4 issue * Fix mypy error expecting bool not np.bool --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
PreviousNext