-
Notifications
You must be signed in to change notification settings - Fork 7
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
Updated resampling resolution comparison #112
base: dswx-ni-beta-point
Are you sure you want to change the base?
Conversation
@@ -1012,8 +1014,14 @@ def run(cfg): | |||
mosaic_cfg = processing_cfg.mosaic | |||
mosaic_mode = mosaic_cfg.mosaic_mode | |||
mosaic_prefix = mosaic_cfg.mosaic_prefix | |||
nisar_uni_mode = processing_cfg.nisar_uni_mode | |||
|
|||
# Determine if resampling is required |
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.
@hb6688 I hope you add more descriptions. nisar_uni_mode
and resamp_required
, and resamp_out_res
are determined from dswx_ni_runconfig.py
, not from user. I think this should be clarified and mentioned here.
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.
Since we've already discussed about the logic to mosaic the images when two RTCs have different spacing and modes, I understand why you implemented like this. However, I hope you add more explanation in the script. It would be easier to read and follow in future when we find the issue.
Co-authored-by: Jungkyo Jung <[email protected]>
Co-authored-by: Jungkyo Jung <[email protected]>
Co-authored-by: Jungkyo Jung <[email protected]>
Co-authored-by: Jungkyo Jung <[email protected]>
Co-authored-by: Jungkyo Jung <[email protected]>
Major Updates:
Updated functions in dswx_ni_runconfig.py
Updated schema and configuration file: