-
Notifications
You must be signed in to change notification settings - Fork 265
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
hyp3
: more metadata for HDF-EOS5
support
#1162
Comments
👋 Thanks for opening your first issue here! Please filled out the template with as much details as possible. We appreciate that you took the time to contribute! |
@yunjunz I agree that supporting HDF-EOS5 files is a great idea -- we're working on a few things on the HyP3 side to provide better metadata overall and effectively eliminate the need of the |
@yunjunz sorry for the slow reply. As @jhkennedy mentioned, we're working on some things to ease the transition from hyp3 -> mintpy. In the meantime however, yep we can add those three attributes to |
Reopen the issue as the |
Hey @yunjunz, the I'll let you know when our team starts in on this work, but right now I'm not sure when that will be. |
What HDF-EOS5 need is the MintPy/src/mintpy/utils/isce_utils.py Lines 203 to 205 in 4d34412
The three metadata ( Here is the related documentation on HDF-EOS5 metadata: https://mintpy.readthedocs.io/en/latest/hdfeos5/#2_metadata |
Description of the desired feature
HDF-EOS5
file format is an easy way to save the final time-series product into a single file. There is now near-automatic support for that after #1161. However, the following metadata is missing. It will be great to provide them from the ASF HyP3 side:Then, all hyp3 products can be saved in HDF-EOS5 format, if the user just turn on the
mintpy.save.hdfEos5
option, without any manual adding of metadata (as I did in here).Let me know what you think please. @forrestfwilliams @talogan @jhkennedy
Next, we would need to update the example ASF HyP3 dataset (to include the above metadata); and cut for a new release of mintpy.
The text was updated successfully, but these errors were encountered: