Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update v.description for S1 data #227

Merged
merged 3 commits into from
Aug 25, 2023
Merged

Update v.description for S1 data #227

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8a25e76
Update v.description for S1 data
jhkennedy Aug 24, 2023
ad8298d
state s1 metadata changes
jhkennedy Aug 24, 2023
95a5e3c
Update CHANGELOG.md
jhkennedy Aug 24, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,12 @@ and this project adheres to [PEP 440](https://www.python.org/dev/peps/pep-0440/)
and uses [Semantic Versioning](https://semver.org/spec/v2.0.0.html).


## [0.11.2]

### Fixed
* Patch [227](hyp3_autorift/vend/CHANGES-227.diff) was applied to align the S1 granules velocity description with the
optical products

## [0.11.1]

### Fixed
Expand Down
15 changes: 15 additions & 0 deletions hyp3_autorift/vend/CHANGES-227.diff
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
diff --git netcdf_output.py netcdf_output.py
--- netcdf_output.py
+++ netcdf_output.py
@@ -639,10 +639,7 @@ def netCDF_packaging(VX, VY, DX, DY, INTERPMASK, CHIPSIZEX, CHIPSIZEY, SSM, SSM1
var = nc_outfile.createVariable('v', np.dtype('int16'), ('y', 'x'), fill_value=NoDataValue,
zlib=True, complevel=2, shuffle=True, chunksizes=ChunkSize)
var.setncattr('standard_name', 'land_ice_surface_velocity')
- if pair_type == 'radar':
- var.setncattr('description', 'velocity magnitude from radar range and azimuth measurements')
- else:
- var.setncattr('description', 'velocity magnitude')
+ var.setncattr('description', 'velocity magnitude')
var.setncattr('units', 'meter/year')
var.setncattr('grid_mapping', mapping_var_name)

4 changes: 4 additions & 0 deletions hyp3_autorift/vend/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,3 +66,7 @@ We've replaced it with `hyp3_autorift.io.get_topsinsar_config`.
from Sentinel-1 pairs that were created using HyP3 autoRIFT versions < 0.9.0, which was released November 2, 2022
9. The changes listed in `CHANGES-223.diff` were applied in [ASFHyP3/hyp3-autorift#223](https://github.com/ASFHyP3/hyp3-autorift/pull/223)
were applied to the S1 correction workflow so that the scene's polarization was set correctly
10. The changes listed in `CHANGES-227.diff` were applied in [ASFHyP3/hyp3-autorift#227](https://github.com/ASFHyP3/hyp3-autorift/pull/227)
were applied to align the S1 granules velocity description with the optical products. These changes have been
[proposed upstream](https://github.com/nasa-jpl/autoRIFT/pull/87) and should be applied in the next
`nasa-jpl/autoRIFT` release.
5 changes: 1 addition & 4 deletions hyp3_autorift/vend/netcdf_output.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -639,10 +639,7 @@ def netCDF_packaging(VX, VY, DX, DY, INTERPMASK, CHIPSIZEX, CHIPSIZEY, SSM, SSM1
var = nc_outfile.createVariable('v', np.dtype('int16'), ('y', 'x'), fill_value=NoDataValue,
zlib=True, complevel=2, shuffle=True, chunksizes=ChunkSize)
var.setncattr('standard_name', 'land_ice_surface_velocity')
if pair_type == 'radar':
var.setncattr('description', 'velocity magnitude from radar range and azimuth measurements')
else:
var.setncattr('description', 'velocity magnitude')
var.setncattr('description', 'velocity magnitude')
var.setncattr('units', 'meter/year')
var.setncattr('grid_mapping', mapping_var_name)

Expand Down
Loading