Skip to content

Conversation

@kgoebber
Copy link
Collaborator

Description Of Changes

This PR adds a couple of additional stand alone examples of various 1D sounding calculations. This also adds a new sounding file for the example that retains the date and location information of the sounding for plotting purposes.

Checklist

@kgoebber kgoebber requested a review from a team as a code owner August 22, 2022 02:58
@kgoebber kgoebber requested review from dcamron and removed request for a team August 22, 2022 02:58
@dopplershift
Copy link
Member

There's a bunch of linting failures still here.

Copy link
Member

@dopplershift dopplershift left a comment

Choose a reason for hiding this comment

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

Thanks for these. Just a few minor tweaks and edits.

@dopplershift dopplershift added Area: Examples Affects examples Type: Enhancement Enhancement to existing functionality labels Sep 8, 2022
@dopplershift dopplershift added this to the August 2022 milestone Sep 8, 2022
@dopplershift
Copy link
Member

Ping @kgoebber don't know if you missed me reviewing this.

@kgoebber
Copy link
Collaborator Author

Yep, somehow missed this one!

Copy link
Member

@dopplershift dopplershift left a comment

Choose a reason for hiding this comment

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

Found a couple typos (and added a couple style suggestions because of it)

Copy link
Member

@dopplershift dopplershift left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks for all the work helping grow our examples!

@dopplershift dopplershift merged commit 30739de into Unidata:main Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Examples Affects examples Type: Enhancement Enhancement to existing functionality

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

More examples in the documentation

2 participants