-
Notifications
You must be signed in to change notification settings - Fork 18
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
Solid Earth Tides correction #91
Merged
Merged
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
348e6c2
Add geometry functionalities
vbrancat 08410be
Add SET functionality
vbrancat 046bc1d
Pep8 notation
vbrancat e9a1221
Keep only LOS SET
vbrancat 8789629
Add SET in CSLC product
vbrancat 2fc5527
Fix metadata allocation for RG SET
vbrancat afa1b28
Add SET component in az direction
vbrancat b7afe0d
Numpy docstring for geometry_utils.py
vbrancat fb774b6
Move open_raster in helper.py
vbrancat 254ea3d
Have rdr2geo outside the SET computation
vbrancat 1595bb4
Compute radar grid using rg_step and az_step from runconfig
vbrancat 2c5953a
Revert "Compute radar grid using rg_step and az_step from runconfig"
vbrancat c28a32e
Revert "Have rdr2geo outside the SET computation"
vbrancat 47e3b1a
Remove azimut SET for testing
vbrancat aaaa181
move rdr2geo computation outside SET
vbrancat 23eae01
pep8 formatting and numpy docstring
vbrancat 4285a58
Merge remote-tracking branch 'upstream/main' into set_test
vbrancat 0f99be1
improve numpy docstring and change division of geometry doppler
vbrancat 56fcb3c
correct sign of heading angle
vbrancat 4cb12f9
Remove datatype return from open_raster function
vbrancat 3b62259
Move ellipsoid params inside rdr2geo function
vbrancat 0e79b52
Return paths to rdr2geo rasters
vbrancat 2405f9f
Simplify rdr2geo raster generation logic, Add check on DEM
vbrancat f6c3d05
Reorganize imports
vbrancat c7bd9f1
Clarify azimuth angle doc string, cite source code
vbrancat 0abe932
Refactor azimuth FM-rate mismatch to use same grid as SET
vbrancat 429ff96
Decimate lat/lon/inc/head for SET computation
vbrancat 33470e0
Resolve conflicts with main branch
vbrancat 7a2f8bc
Push complete list of reqs for CI unit test
vbrancat f2204a9
docker use s1-reader main
LiangJYu 94eae40
Merge pull request #6 from LiangJYu/set_test
vbrancat c62a36f
explicit call out slant range SET
LiangJYu e2425b5
less repitition with rdr2geo raster load
LiangJYu 3a12cea
less repitition in SET computation
LiangJYu 1fd4888
nit - add space
LiangJYu f1d78b0
fix typo
LiangJYu f19d73c
fix raster order
LiangJYu 0f85cce
Merge pull request #7 from LiangJYu/set_test
vbrancat File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
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.
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.
This is better for CI to take the changes in
s1-reader
in to account, but I think we'll need to hard code its version in the upcoming deliveries. @vbrancat @LiangJYu Any opinions?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.
is there any reason not to put s1reader on pypi so we can just pin the version like other requirements?