v0.4.0 release: visualization module and orbital parameter search #204
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add geoviews as extra optional dependency in setup.py * Setup miniconda and conda install proj in .travis.yml file Using conda to install PROJ, so that cartopy can be installed, and therefore geoviews can be installed too! * Pin to PROJ 7.2.0 Perhaps cartopy doesn't work with PROJ 8.0.0 yet? * Have Travis CI install geos into icepyx-env * Set minimum cartopy version to 0.18.0 * Update setup.py Co-authored-by: Tian Li <[email protected]> Co-authored-by: Tian Li <[email protected]>
…176) Co-authored-by: Wei Ji <[email protected]> Co-authored-by: Jessica Scheick <[email protected]>
Rename all icepyx references of the 'master' git branch to 'main'. Co-authored-by: Wenjie Ji <[email protected]>
new functionality to search by temporal or orbital parameters improvements to formatting and management of request parameters (use strings to prevent url formatting) decode response XML if verbose update tests and documentation for orbital parameter searches Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: Wei Ji <[email protected]>
* Ensure API docs are generated to gitignored doc/source/_icepyx folder * Fix whitespace issues in *.rst files * Fix whitespace issues in Earthdata.py and variables.py Adding or removing spaces so that Sphinx stops complaining like "WARNING: Explicit markup ends without a blank line".
* Added Conda &PyPi install badges to Github homepage * Added conda and pip badges * reverse previous commit by removing badges * added conda and pip badges
…#195) * surface query error messages returned from NSIDC * catch no subset data messages and alert user * add query exceptions module and use in granules search
* generalize ICESat-2 product for OA API request and create a seperate visualization module * Remove invaid OA request, use dask and xarray to speed up datamanipulation, enhance interactive visualization based on cycle number * Update requirements.txt Co-authored-by: Wei Ji <[email protected]> * change typos and formatting Co-authored-by: Wei Ji <[email protected]> * update notebook * Improve visualization - Add product check - Increase number of concurrent multithreading max_worker to reduce API requesting time - Use backoff to deal with specfic exceptions raised during API request - Set elevation sampling rate of 1/20 and cycle number of latest cycle to optimize memory and speed up data processing - Add testing to all supported products - Add an interactive map to visualize elevation spatial distribution and along-track elevation for individual RGT - Update visualization notebook * remove output from first cell * Update icepyx/core/visualization.py Co-authored-by: Wei Ji <[email protected]> * Apply suggestions from code review Co-authored-by: Wei Ji <[email protected]> * remove visualize_extent func from visualization.py * add rgt/cycle in Visualization * run black to format code and alphebetize imports * add visualization module to top-level import * use pd.json_normalize to clean data * Revert "use pd.json_normalize to clean data" This reverts commit 81a9fc0. * use pd.json_normalize, add sample_rate parameter, use dask dataframe instead of xarray.dataset * add visualization_elevation function in Query class to call Visualize * Update icepyx/tests/test_visualization.py Co-authored-by: Wei Ji <[email protected]> * fix typo * remove geoviews from Visualize and print statement from Query.visualize_elevation * add tests for orbital query object, add user control for visualization * Update icepyx/core/visualization.py Co-authored-by: Wei Ji <[email protected]> * Update icepyx/core/visualization.py Co-authored-by: Wei Ji <[email protected]> * Update icepyx/core/visualization.py Co-authored-by: Wei Ji <[email protected]> * Apply suggestions from code review Co-authored-by: Wei Ji <[email protected]> * remove old dev notebooks * use query object in visualize __init__ to allow multiple access options and use built-in checks * add OpenAltimetry valid product check * add visualization to docs and add docstrings; reformat with black * update test for new visualization init * add error types * visualization example updates * remove top-level visualization module import and update example accordingly * fix variable names in test * add IndexError * add a test * add function gran_paras based on gran_IDs, add test * Revert "remove old dev notebooks" This reverts commit 53b55c4. * wrap gran_paras vis function into granules gran_IDs * remove print statements * add contributors to notebook Co-authored-by: Wei Ji <[email protected]> Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: Jessica <[email protected]>
Co-authored-by: Wei Ji <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #204 +/- ##
==========================================
+ Coverage 48.17% 55.63% +7.45%
==========================================
Files 17 20 +3
Lines 1179 1517 +338
Branches 262 316 +54
==========================================
+ Hits 568 844 +276
- Misses 570 616 +46
- Partials 41 57 +16
Continue to review full report at Codecov.
|
weiji14
approved these changes
May 13, 2021
JessicaS11
commented
May 13, 2021
Co-authored-by: Wei Ji <[email protected]>
|
FYI, the conda-forge autobot has opened up conda-forge/icepyx-feedstock#2 for icepyx v0.4.0, I've updated the dependencies (and included |
|
Thanks for taking care of this @weiji14! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Lots of great work included here that we want to make available via conda and pip: