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.

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

GAMMA handling improvements #245

Merged
merged 8 commits into from
May 4, 2023
Merged

GAMMA handling improvements #245

merged 8 commits into from
May 4, 2023

Conversation

johntruckenbrodt
Copy link
Owner

@johntruckenbrodt johntruckenbrodt commented Apr 30, 2023

This enhances the usability of the gamma module:

  • GAMMA's LAT module is no longer needed: new pyroSAR-internal implementations can be used if the module is missing (concerns commands product, ratio and linear_to_dB)
  • improved backwards compatibility: new cases have been added in which pyroSAR uses the newest command if usable and falls back to an older alternative if not (multi_look_ScanSAR->multi_S1_TOPS, gc_map2 does not provide all needed arguments); an additional check addresses the case that gc_map might not have an argument OFF_par
  • gamma.pixel_area_wrap: new argument exist_ok to run the function in update mode; eventually this argument will be introduced in gamma.geocode as well
  • various bug fixes
    • gamma.convert2gamma: raise an error if S1_bnr=True but the GAMMA command does not support border noise removal
    • gamma.geocode: removed unneeded underscore in HDR file naming
    • examine.ExamineGamma: search for gdal-config command instead of a hard-coded path
    • gamma.pixel_area_wrap: fixed some issues with occasionally missing intermediate files, e.g. for computing ratios
  • documentation improvements

@johntruckenbrodt johntruckenbrodt merged commit 3c6deb2 into main May 4, 2023
@johntruckenbrodt johntruckenbrodt deleted the feature/gamma branch May 4, 2023 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant