GeoscienceAustralia / PyRate

A Python tool for estimating velocity and time-series from Interferometric Synthetic Aperture Radar (InSAR) data.
https://geoscienceaustralia.github.io/PyRate/
Apache License 2.0
199 stars 69 forks source link

Metadata addition: reference pixel coordinates to metadata in output files in velocity directory #358

Closed adeane-ga closed 2 years ago

adeane-ga commented 2 years ago

Problem Outside users of PyRate products often only engage with or have access to the results from the velocity directory velocity_dir. The files in there are:

None of these files have the reference pixel coordinates in the GeoTIFF metadata.

For any practical interpretation of PyRate results, a user needs to know where the reference pixel is.

Currently it is within the metadata of the time series files (tscuml_YYYYMMDD-YYYYMMDD.tif files in timeseries_dir), but practically users may only have access to the ultimate velocity results and not engage with time series data.

Request Simply place the reference pixel coordinates into the GeoTIFF metadata of the files that are output into the velocity directory (like it has been for the time series GeoTIFFs).

mcgarth commented 2 years ago

I believe the reference pixel coordinates are already saved to the interferogram geotiff metadata, in both X/Y and lat/lon notation.

adeane-ga commented 2 years ago

I believe the reference pixel coordinates are already saved to the interferogram geotiff metadata, in both X/Y and lat/lon notation.

This is a strange one, because reference pixel metadata is not currently being saved to the GeoTIFFs in the velocity directory. But in some past solutions it has been. So I am not sure what's changed, and I cannot find the section of code that does add reference pixel metadata to the velocity directory files - this current PR is working on it https://github.com/GeoscienceAustralia/PyRate/pull/369.

But, yes the reference pixel metadata is currently being saved to the interferogram and time series GeoTIFFs. This addition is for users who are only looking at the velocity directory files.

adeane-ga commented 2 years ago

Closing issue because it was fixed with this merge into develop: https://github.com/GeoscienceAustralia/PyRate/pull/369