erdc / quest

Python API for downloading and managing data. Checkout the documentation at:
https://quest.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
14 stars 9 forks source link

Workflows #111

Closed sdc50 closed 6 years ago

sdc50 commented 6 years ago

Testing notebooks

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 418


Files with Coverage Reduction New Missed Lines %
/Users/travis/miniconda/envs/test-environment/lib/python3.5/site-packages/quest/util/config.py 3 89.55%
/Users/travis/miniconda/envs/test-environment/lib/python3.5/site-packages/quest/util/param_util.py 21 69.05%
/Users/travis/miniconda/envs/test-environment/lib/python3.5/site-packages/quest/util/misc.py 34 76.51%
/Users/travis/miniconda/envs/test-environment/lib/python3.5/site-packages/quest/api/datasets.py 60 54.85%
<!-- Total: 118 -->
Totals Coverage Status
Change from base Build 387: -0.2%
Covered Lines: 1526
Relevant Lines: 2193

💛 - Coveralls
AaronV77 commented 6 years ago

@sdc50 why is the WMTS notebook being added to this PR?

sdc50 commented 6 years ago

@AaronV77 I've made changes to the notebooks to leverage the new workflows interface. I've also made running the notebooks part of the tests. I'm just trying to get everything working on CI so it's ready to merge after PR #110 get merged in.

AaronV77 commented 6 years ago

@sdc50 alright that makes sense. Can you make sure that the notebook(s) output are cleared out. Thanks

sdc50 commented 6 years ago

@AaronV77 This is ready for review. I just pulled the usgs_ned:19-arc-second tests out since the service isn't being consistent right now.