-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Yanked] Release: v0.2.0 #424
Merged
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
Fix levels
change name of variable (los_file -> lookvector) fix masking with a reshape make shape of outputs consistent (LOS file, orbit file)
Fix buffer raytracing
add variable to context manager so hgts are retrievied
* add jupyter to conda install * add wand for pdf viewing in jupyter * check for heights in actual dataframe * fix bug in writing delays for gnss station file * correct output filename for stations * automatically change file extension to nc if not nc or h5 * force rasterio version to 3.3 * make rasterio >= 1.3.3
#401) * begin support for cli interface to dlGNSS and specific steps for delay * doStep is now supported and enables GNSS download * doStep now fully works * download GNSS stations works * change name of gps output dir
Co-authored-by: Jeremy Maurer <[email protected]>
* support reading lat/lons from a GUNW * enable calculation of delay with a GUNW file and specified height levels * extra check to make sure GNSS file is treated separately * update test to reflect change in output of rio_extent (from WSEN to SNWE) * match type for updated test * enable reading of bbox from ISCE geocoded lat/lon files * calc delays at DEM heights * remove accidentally addited GNSS cli
* validator bug fix * support for reusing weather model file in projected coords (HRRR) * remove an if condition and extra comments
* make sure the right steps happens and default to only weather model calculation * Add dem download if hgt file not defined for latlon inputs * fix a bounding box bug for GMAO * fix bounding box problem and remove broken models * Update README.md Does not need the env to create it the first time. * Update README.md Adding Piyush to the README as contributor * Fix some parsing and extent checking bugs * Update processWM.py update log message * Update runProgram.py fix for showing debug messages correctly * Update logger.py correctly show debug messages when user requests verbose Co-authored-by: Jeremy Maurer <[email protected]> Co-authored-by: dbekaert <[email protected]> Co-authored-by: BB <[email protected]>
* change CLI to raider.py and fns to main * fix fn name in tests * update circleci; start fixing old functions in downloadGNSS * replace raiderDelay with raider everywhere * downloadGNSS notebook and CLI (dostep) works * relocate downloadGNSS and make it work * bugfix for reading weather files * remove steps functionality * add option for only downloading weather model * put back files git deleted
Fix the TOC and hyperlinking.
* temporary gdal fix * cleanup circleci config and switch to micromamba * ensure micromamba is available * constrain isce3 build * try revert gdal pin * Constrain python version for version info workflow * try * in buildstring Co-authored-by: Jeremy Maurer <[email protected]>
* update unit tests * update unit tests and add end-to-end tests * make end-to-end tests long * add the orbit file * take out conventional slant for now * golden unit tests * add golden datasets * update golden datasets paths * move files Co-authored-by: Jeremy Maurer <[email protected]>
* Add release automation * Update CHANGELOG.md fix conventional to not be supported Co-authored-by: dbekaert <[email protected]>
* remove long test flags * Skip CDSAPI test for now, passing locally but failing for CircleCI Co-authored-by: Jeremy Maurer <[email protected]>
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Closed
@dbekaert @jhkennedy you can edit the PR description if needed. |
jhkennedy
approved these changes
Dec 3, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
EDIT: This was "yanked" as it was merged using a squash merge, which screws up the assumptions with
setuptools_scm
and the release workflow