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

SAR: Req. category 1.6 ambiguous and unnecessary? #26

Open
m-mohr opened this issue May 2, 2024 · 6 comments
Open

SAR: Req. category 1.6 ambiguous and unnecessary? #26

m-mohr opened this issue May 2, 2024 · 6 comments
Labels

Comments

@m-mohr
Copy link
Contributor

m-mohr commented May 2, 2024

The "category" 1.6 in the Radar PFS seems to describe a requirement, but it's unclear whether it's a minimum or goal requirement:

Note: Source data attribute information are described for each acquisition and sequentially identified as acqID= 1, 2, 3, …

While I can easily make the attribute information available for each acquisition through derived_from links in STAC, there's no standardized way to specify an id that follows the 1, 2, 3, ... numeration approach.

Also, I'm wondering isn't this implicitly define by the acquisition timestamps?
Let's say I link to all acquisitions and they have distinct timestamps associated with them, shouldn't that be enough to create a sequence of images? Do we really need to enfore the 1,2,3 numbeing? It's fine if that's required in the XML spec, but it would be great if I don't need to do that in STAC.

cc @akerosenqvist

@m-mohr m-mohr changed the title Radar: Req. category 1.6 ambiguous Radar: Req. category 1.6 ambiguous and unnecessary? May 2, 2024
@libbyrose libbyrose added the SAR label May 3, 2024
@akerosenqvist
Copy link
Collaborator

akerosenqvist commented May 3, 2024

Item 1.6 "Source Data Attributes" and item 1.7 "CEOS-ARD Product Attributes" is the way the SAR PFS accommodates multi-source products.

In the case a CEOS-ARD product is generated from more than one data source (such as a mosaic product), the parameters 1.6.1 - 1.6.12 will need to be provided for each of the source data. The same parameter names are thus repreated several times (as many times as there are datatakes) in the metadata file and the "acqID" parameter is therefore required to indicate which datatake a given parameter refers to (so in this sense, it is indeed a Threshold requirement).
See example here: https://www.dropbox.com/scl/fi/0u93aqhh4lcyvu205xxbg/N83W080_2023_F02DAR.xml?rlkey=vk6zb3t3b8it80lwafafurmmr&dl=0

How you resolve this in the STAC extension, I'd need to understand the alternatives better in order to have an opinion. I've copied in John again.

CC: @johntruckenbrodt

@m-mohr
Copy link
Contributor Author

m-mohr commented May 3, 2024

@akerosenqvist That's understood, thanks.

In STAC the Source data + metadata is located in other STAC Items that the product links to. The product STAC Item marks then as "derived from" the following source(s) ... It doesn't make sense in STAC to number then. That an encoding specific that should be in the XML metadata spec, I think. acqID is not defined in the PFS, it's an XML encoding detail. In STAC you can derive the sequence from the timestamps of the individual STAC Items of the source data. I believe that should work and give the same order as just 1,2,3,... , but the PFS doesn't allow it but insists on numbers.

I think the quote above should be split into two pieces:

PFS PDF:

Note: Source data attribute information are described for each acquisition and must idenfity the sequence of the acquisitions, e.g. through the acquisition timestamps or an enumerated list.

XML Spec:

Note: The source data for each acquisition must be sequentially identified as acqID= 1, 2, 3, …

or something similar.

@m-mohr m-mohr changed the title Radar: Req. category 1.6 ambiguous and unnecessary? SAR: Req. category 1.6 ambiguous and unnecessary? May 3, 2024
@m-mohr
Copy link
Contributor Author

m-mohr commented May 3, 2024

I guess I found the place why the acqID is as it is: It's in req 2.8. But there it actually also seems to allow day or time references instead of ID...

@akerosenqvist
Copy link
Collaborator

akerosenqvist commented May 3, 2024

Adding @francoischarbonneau to the discussion

@johntruckenbrodt
Copy link

@m-mohr, exactly, the ID was meant to correspond to the numbering in 2.8. Perhaps just describing in 2.8 that the numbering corresponds to the list of source images sorted by acquisition time in ascending order would be enough.

@strobpr
Copy link

strobpr commented May 26, 2024

I believe techniques such as 'mosaicking' or 'data fusion' are relevant beyond SAR and should not be buried here. The real issue I see behind this is that the distinction between 'general' and 'per pixel' metadata, which we so far have prominently in all PFSs (afaik), becomes inadequate as soon as we allow 'mosaicking', starting with different acquisition times, to later different sensors and missions.

We should have this discussion, but for that we need a better categorisation of product types and/or processing levels and what they exactly entail.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

5 participants