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

Added sat:anx_start_offset and sat:anx_end_offset fields #9

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

m-mohr
Copy link
Contributor

@m-mohr m-mohr commented May 22, 2024

Any thoughts?

Was discussed with ESA for Sentinel-1. I changed from "type integer, unit nanoseconds" to type floating point number, i.e. flexible unit. floating point numbers can also be added / subtracted more easily from unix timestamps etc.

@emmanuelmathot
Copy link
Member

emmanuelmathot commented May 22, 2024

Those offsets represent the delays between crossing the time at crossing the ANX and the start/end of the acquisition, right? At that point those 2 values can be easily derived by substracting the sat:anx_datetime from start_datetime and end_datetime. That is why I did not put them initially in this extension.

@m-mohr
Copy link
Contributor Author

m-mohr commented May 22, 2024

I hope ESA can clarify, I'm not sure as I didn't follow the full length of the discussion. If that is the case though, we should probably not add them here, I agree.

Edit: ESA agreed to provide the usecase and an example query that requires such offsets.

@jolyonm
Copy link

jolyonm commented Jun 17, 2024

The “old” requirement for many users of SAR was to find matching sets of data according to relative orbit and a known offset from the anx

Whilst the offset is certainly calculatable from the sensing time – anx crossing it may not be an easy query to manage via the catalogue
e.g. where relative_orbit = myOrbit AND sensing_start – anx_datetime < myInterestEnd AND senting_stop – anx_datetime > myInterestStart
or something similar, implying a date calculations on the server side

The query is of course equivalent to expressing an AOI, but now I’m not sure if all legacy missions reliably support AOI for the RAW data catalogues. Perhaps this “old” requirement should be retired, but it seems that if we want to report anx_datetime per item it makes equal sense to report the start_offset and end_offset as optional fields

@m-mohr m-mohr self-assigned this Jun 17, 2024
@emmanuelmathot
Copy link
Member

@jolyonm this use case can be covered using the existing sat:anx_datetime field.

@m-mohr
Copy link
Contributor Author

m-mohr commented Jul 22, 2024

The query is of course equivalent to expressing an AOI, but now I’m not sure if all legacy missions reliably support AOI for the RAW data catalogues.

This should be clarified first. All data needs to be on the table before we add something here.

Perhaps this “old” requirement should be retired, but it seems that if we want to report anx_datetime per item it makes equal sense to report the start_offset and end_offset as optional fields

This seems like very niche information that can be computed. Storing it in the database (with indexed for querying) seems like a lot of overhead for such a case. I think we should avoid this if possible.

Setting to draft until we have all information provided.

@m-mohr m-mohr marked this pull request as draft July 22, 2024 11:49
@m-mohr m-mohr removed their assignment Jul 22, 2024
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.

3 participants