pysat / pysatMadrigal

Madrigal instrument support for the pysat ecosystem
BSD 3-Clause "New" or "Revised" License
1 stars 1 forks source link

ENH: updates GNSS tec documentation, methods testing #58

Closed aburrell closed 2 years ago

aburrell commented 2 years ago

Description

Improves the GNSS methods testing, adds a TEC example, and expands general Madrigal metadata definitions. Also fully defines the GNSS VTEC metadata. Partially addresses #3 and #41.

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Created an example in the docs and added unit tests. You should also now receive no warning when loading VTEC data.

Test Configuration

Checklist:

If this is a release PR, replace the first item of the above checklist with the release checklist on the pysat wiki: https://github.com/pysat/pysat/wiki/Checklist-for-Release

rstoneback commented 2 years ago

Not required, and it probably makes the example too complicated for the docs, but it would be nice if we could get coastlines on the VTEC plot as well.

aburrell commented 2 years ago

Yeah, I decided against it because I didn't want to figure out carteopy or get carteopy questions in the future 🗺️