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

Focusing quality improvements #2

Open
pr2718 opened this issue Sep 5, 2023 · 8 comments
Open

Focusing quality improvements #2

pr2718 opened this issue Sep 5, 2023 · 8 comments

Comments

@pr2718
Copy link
Collaborator

pr2718 commented Sep 5, 2023

  • Secondary range compression(TBD if needed)
  • Range/Azimuth compression windowing
  • Vr & Doppler centroid changes in azimuth direction
@kautlenbachs
Copy link
Member

The TDS01 ERS1 example dataset has range size 5000 for current development version, but baseline has 4991, hence a 5 pixel padding exclusion.

@kautlenbachs
Copy link
Member

TDS09 has 5128 vs 5141, hence a 6/7 padding.

@kautlenbachs
Copy link
Member

Focussing quality should also improve for the areas not in the middle of picture (maybe need to modify usage of Vr as well).

Echo metadata might need to be propagated as well, because the parser does not know
a) how much of an extra lines should be parsed (besides sensing start and end)
b) when windowing range and azimuth, the metadata shall be updated

@kautlenbachs
Copy link
Member

There is something interesting happening regarding SWST for ERS here - https://github.com/gmtsar/gmtsar/blob/master/preproc/ERS_preproc/ers_line_fixer/ers_line_fixer.c#L434

@kautlenbachs
Copy link
Member

One option would be to simply pad left and right for the range as it is done here - https://github.com/isce-framework/isce2/blob/main/components/isceobj/Sensor/ERS_EnviSAT.py#L557 This would be instead of cutting range.

@kautlenbachs
Copy link
Member

Investigate sar_metadata.h CalcKa() and CalcAperturePixels(). Further on investigate L1 datasets which L0 has not a long sensing time -> therefore try to find how much was cut by the reference processor (because it cannot include more pixels in the beginning and in the end and then should publish less data than requested by the sensing end and start?!). Try to correlate these findings with CalcAperturePixels()

@kautlenbachs
Copy link
Member

kautlenbachs commented Jan 3, 2024

Results windowing is implemented with this pull request - #33

@kautlenbachs
Copy link
Member

Range compression windowing is implemented by this PR - #34

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

No branches or pull requests

2 participants