-
Notifications
You must be signed in to change notification settings - Fork 6
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
More changes after the calval release #52
Conversation
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.
Looks like the most of the changes in this PR are to deal with the features merged in ISCE3 side. The PR looks good to me.
0e02be9
to
2f4a3a9
Compare
…C; save noDataValue=NaN for float data
…o the product metadata
…ts when flag_process is false
…te runconfig comments (2)
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.
Nice work. Here are my comments and suggestions - Seongsu
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.
LGTM. Thanks for the addressing the comments.
Please rebase the PR #55 after merging this one, so that I can have a better look at the following PR.
Changes: