Skip to content
This repository was archived by the owner on Jan 9, 2020. It is now read-only.

Conversation

deeplycloudy
Copy link

Gridded GLM imagery are now available on threads-test. This example adapts the water vapor + aircraft example to overlay GLM data on ABI imagery. Requires Unidata/siphon/#272 for parsing times.

@deeplycloudy
Copy link
Author

Pinging this PR now that siphon/#272 has been merged.

@zbruick
Copy link
Contributor

zbruick commented Aug 14, 2019

Yes, this deserves reviewing now, but it won't pass Travis until Siphon gets a new release.

Copy link
Contributor

@zbruick zbruick left a comment

Choose a reason for hiding this comment

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

Overall, this looks really good. While it will be one of our more complex examples, I'm not sure how to reduce it down any to still get what we want. The only thing I'm curious about adding is a colorbar for the GLM data. If that's not too much to ask, and doesn't make the plot too busy, I think that could be useful.

deeplycloudy and others added 2 commits August 24, 2019 09:28
Co-Authored-By: Zach Bruick <zachary.bruick@valpo.edu>
Copy link
Contributor

@zbruick zbruick left a comment

Choose a reason for hiding this comment

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

This looks good, but we'll have to sit on it until the next siphon release for CI to pass.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants