From f72c33356344939e83dbb0b1cd320882fa12eb39 Mon Sep 17 00:00:00 2001 From: Robert Jackson Date: Wed, 8 Apr 2026 12:57:09 -0500 Subject: [PATCH 1/2] FIX: Add missing units attr and correct swapped long_name on u/v/w fields The u, v, w DataArrays returned by get_dd_wind_field were missing the 'units' attribute entirely. Also, u_field and v_field had their long_name values swapped (u is zonal, v is meridional). Fixed in both the scipy and tensorflow output paths. Co-Authored-By: Claude Sonnet 4.6 --- pydda/retrieval/wind_retrieve.py | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/pydda/retrieval/wind_retrieve.py b/pydda/retrieval/wind_retrieve.py index 5a5a0a2f..0e0a4c34 100644 --- a/pydda/retrieval/wind_retrieve.py +++ b/pydda/retrieval/wind_retrieve.py @@ -748,17 +748,20 @@ def loss_and_gradient(x): u_field = {} u_field["standard_name"] = "u_wind" - u_field["long_name"] = "meridional component of wind velocity" + u_field["long_name"] = "zonal component of wind velocity" + u_field["units"] = "m/s" u_field["min_bca"] = min_bca u_field["max_bca"] = max_bca v_field = {} v_field["standard_name"] = "v_wind" - v_field["long_name"] = "zonal component of wind velocity" + v_field["long_name"] = "meridional component of wind velocity" + v_field["units"] = "m/s" v_field["min_bca"] = min_bca v_field["max_bca"] = max_bca w_field = {} w_field["standard_name"] = "w_wind" w_field["long_name"] = "vertical component of wind velocity" + w_field["units"] = "m/s" w_field["min_bca"] = min_bca w_field["max_bca"] = max_bca @@ -1270,17 +1273,20 @@ def _get_dd_wind_field_tensorflow( u_field = {} u_field["standard_name"] = "u_wind" - u_field["long_name"] = "meridional component of wind velocity" + u_field["long_name"] = "zonal component of wind velocity" + u_field["units"] = "m/s" u_field["min_bca"] = min_bca u_field["max_bca"] = max_bca v_field = {} v_field["standard_name"] = "v_wind" - v_field["long_name"] = "zonal component of wind velocity" + v_field["long_name"] = "meridional component of wind velocity" + v_field["units"] = "m/s" v_field["min_bca"] = min_bca v_field["max_bca"] = max_bca w_field = {} w_field["standard_name"] = "w_wind" w_field["long_name"] = "vertical component of wind velocity" + w_field["units"] = "m/s" w_field["min_bca"] = min_bca w_field["max_bca"] = max_bca From 9264931a4e4548965a3f4b7713bf29bc6a4fd731 Mon Sep 17 00:00:00 2001 From: Robert Jackson Date: Wed, 8 Apr 2026 12:57:46 -0500 Subject: [PATCH 2/2] REL: Bump version to 2.4.1 Co-Authored-By: Claude Sonnet 4.6 --- pydda/__init__.py | 2 +- setup.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/pydda/__init__.py b/pydda/__init__.py index d96cd4c3..04163bd7 100644 --- a/pydda/__init__.py +++ b/pydda/__init__.py @@ -12,7 +12,7 @@ from . import constraints from . import io -__version__ = "2.4.0" +__version__ = "2.4.1" print("Welcome to PyDDA %s" % __version__) print("If you are using PyDDA in your publications, please cite:") diff --git a/setup.py b/setup.py index 0252d678..f8db0810 100644 --- a/setup.py +++ b/setup.py @@ -42,7 +42,7 @@ PLATFORMS = "Linux, Windows, OSX" MAJOR = 2 MINOR = 4 -MICRO = 0 +MICRO = 1 # SCRIPTS = glob.glob('scripts/*') # TEST_SUITE = 'nose.collector'