Skip to content

ADD: Harr wavelet example - #1041

Open
rcjackson wants to merge 2 commits into
ARM-DOE:mainfrom
rcjackson:harr_wavelet
Open

ADD: Harr wavelet example#1041
rcjackson wants to merge 2 commits into
ARM-DOE:mainfrom
rcjackson:harr_wavelet

Conversation

@rcjackson

Copy link
Copy Markdown
Collaborator

This PR implements the Harr wavelet technique for retrieving PBL from lidar backscatter profiles. An example for the test ceilometer data is included. Co-authored by Claude Sonnet 5. This will add PyWavelets as an optional dependency.

References:

Brooks, I. M. (2003). Finding boundary layer top using
wavelet covariance transform. Journal of Atmospheric and Oceanic
Technology, 20(8), 1092-1105.
https://doi.org/10.1175/1520-0426(2003)20%3C1092:FBLTUB%3E2.0.CO;2

Cohn, S. A., & Angevine, W. M. (2000). Boundary layer height and
entrainment zone thickness measured by lidars and wind-profiling
radars. Journal of Applied Meteorology, 39(8), 1233-1247.
https://doi.org/10.1175/1520-0450(2000)039%3C1233:BLHAEZ%3E2.0.CO;2

  • Closes #xxxx
  • Tests added
  • Documentation reflects changes
  • PEP8 Standards or use of linter
  • Xarray Dataset or DataArray variable naming follows 'ds' or 'da' naming

Comment thread act/retrievals/pbl_lidar.py Outdated


def calculate_wavelet_pbl(
in_ds,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for consistency, can we keep this as 'ds'

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in latest commit

@zssherman

Copy link
Copy Markdown
Collaborator

@AdamTheisen Are you fine if we add a new optional dependency for this code?

@AdamTheisen

Copy link
Copy Markdown
Collaborator

@zssherman I think a new dependency like scipy is fine to bring in and likely something we would do in the long run anyways. I would want to finalized a decision on my question in #1040 before merging this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants