Skip to content

Conversation

@abradley60
Copy link
Collaborator

  • Correcting the platform filename bug. i.e. sentinel 1B products were fixed as s1a in the product name
  • Bugix for linking static layers. The layers to link are now explicitly set as a list. e.g. ["number_of_look","gamma0_to_beta0_ratio" ...]
  • Various updates to XML metadata:
    • <PerformanceIndicators pol="HH”> -> <PerformanceIndicators pol="VV,VH,HH,HV">
    • <SourceProcParam/ProcessingDate -> append X for UTC time
    • <ProductENL> -> removed
    • Adding NumberOfLooks and GammaToBetaRatio as </PerPixelMetadata
    • expand <GriddingConvention :
      <GriddingConvention> <BurstGeogridSnapX units="m"> </BurstGeogridSnapX> <BurstGeogridSnapY units="m"> </BurstGeogridSnapY> <GriddingConventionReference type="URL"> </GriddingConventionReference> </GriddingConvention>

Copy link
Collaborator

@caitlinadams caitlinadams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just checking one thing with the types in the XML

CEOS-ARDProductAttributes/PerPixelMetadata/GammaToBetaRatio/ByteOrder,FIXED_VALUE,Little Endian,,
<NumberOfLooks>,,,,
CEOS-ARDProductAttributes/PerPixelMetadata/NumberOfLooks/FileName,STAC,assets.number_of_looks.href,,
CEOS-ARDProductAttributes/PerPixelMetadata/NumberOfLooks/SampleType,FIXED_VALUE,Ratio,Count,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this correct that this has both Ratio and Count? Should it have Count only, or count twice?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great pickup thanks Caitlin. Unit for NumberOfLooks should be count.

@abradley60
Copy link
Collaborator Author

@caitlinadams just implemented a bugfix as the xml was giving errors for runs when STATIC_LAYERS aren't linked. I've added a REQUIRED column to the mapping csv. Are you OK to review?

@abradley60 abradley60 merged commit 1e659d8 into main Sep 5, 2025
2 checks passed
@caitlinadams caitlinadams deleted the updates/xml_fix branch September 5, 2025 03:16
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.

2 participants