pysat / pysatNASA

pysat support for NASA Instruments
BSD 3-Clause "New" or "Revised" License
21 stars 6 forks source link

BUG/MAINT: cdaweb downloads, ICON IVM-B tests, new flag to skip new tests #223

Closed jklenzing closed 8 months ago

jklenzing commented 11 months ago

Description

Addresses #218 (partial)

Fixes a major issue where cdasws ignores the end date for download requests if no data is present at 00:00:00 on the end date. This fixes most of the items in #218.

Adds testing flags for upcoming pysat pull to ignore "new" tests for now while we update the ecosystem. Usage of flags includes TODO statements linked to the relevant issue.

Turns on tests for ICON IVM-B now that some data is public for that.

Type of change

How Has This Been Tested?

pytest using the enh/test_triage branch of pysat

Test Configuration:

Checklist:

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

jklenzing commented 10 months ago

Tests against pysat RC here: https://github.com/pysat/pysatNASA/actions/runs/7575679334

jklenzing commented 9 months ago

TODO:

jklenzing commented 8 months ago

On doc failure: SABER has added a human check to their websites. Moving the link to the comments so that the link check will skip this.

aburrell commented 8 months ago

Check the fix I made on the GUVI branch, it's a bit more "smart"