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

Some burst border is larger than the actual coverage #124

Open
seongsujeong opened this issue Jul 22, 2023 · 1 comment
Open

Some burst border is larger than the actual coverage #124

seongsujeong opened this issue Jul 22, 2023 · 1 comment
Labels
bug Something isn't working needs triage Issue requires triage to proceed

Comments

@seongsujeong
Copy link
Contributor

seongsujeong commented Jul 22, 2023

Checked for duplicates

Yes - I've already checked

Describe the bug

During the ANX testing, it was found that some bursts' border attribute has polygons protruding outside the area of the bounding box defined in burst DB. After investigation, I suspect some incorrect latitude / longitude values in geolocationGridPointList, from which Sentinel1BurstSlc.border is computed.
Below is the example data with issue.

SAFE: S1A_IW_SLC__1SDV_20200101T171339_20200101T171405_030610_0381C6_5514.zip
ORBIT: S1A_OPER_AUX_POEORB_OPOD_20210316T161714_V20191231T225942_20200102T005942.EOF
Burst ID: t088_186933_iw2. This is the 9th burst in the IW2 subswath in VV polarization.
Polarization: VV

Below is the plot of each burst borders in IW2 VV polarization. x / y values in the plot are longitude / latitude respectively. The burst mentioned above is labeled as 9 / 9, meaning that it is the 9th burst In the subswath.
Screenshot 2023-07-22 at 01 47 33

Below is how the 9th border looks like when it is reprojected and overlaid on the corresponding bounding box. The red polygon is the burst border, and the blue one is the bounding box from burst DB.
Screenshot 2023-07-21 at 18 48 08

Below is how the screenshot of the RTC product (VV pol.) suggesting that the coverage of the actual burst is well within the bounding box in burst DB.
Screenshot 2023-07-22 at 02 12 37

What did you expect?

I think we need an algorithm to take care of the cases when incorrect longitude / latitude information in is in geolocationGridPointList. Also it would be necessary that any downstream workflows that make use of Sentinel1BurstSlc.border needs to be aware of this issue.

Reproducible steps

The test dataset mentioned above is available upon request.

Environment

- Version of this software [e.g. vX.Y.Z]
- Operating System: [e.g. MacOSX with Docker Desktop vX.Y]
...
@seongsujeong seongsujeong added bug Something isn't working needs triage Issue requires triage to proceed labels Jul 22, 2023
@seongsujeong seongsujeong changed the title Burst boundary larger than its actual coverage Burst border larger than its actual coverage Jul 22, 2023
@seongsujeong seongsujeong changed the title Burst border larger than its actual coverage Some burst border is larger than the actual coverage Jul 22, 2023
@seongsujeong
Copy link
Contributor Author

seongsujeong commented Jul 25, 2023

S1B_IW_SLC__1SDV_20210926T231303_20210926T231331_028876_037232_1AE7 SAFE-t150_320052_iw1

The plot above is another type of case that SentinelBurstSlc.border gets protruded from the corresponding bounding box. Like mentioned above, the error in the polygon does not seem to affect the actual coverage of the SLC. However, the users who are making use of this polygon need to be aware of this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage Issue requires triage to proceed
Projects
None yet
Development

No branches or pull requests

1 participant