Skip to content

Conversation

@abradley60
Copy link
Collaborator

Changes

  • AUS_COP_HUB added as an option for downloaded the source scene
    • AUS_COP_HUB credentials added to example .env file
    • Download from AUS_COP_HUB requires the pygssearch package. Unfortunately, this cannot be installed in the sar-pipeline environment due to numerous package conflicts that break the workflow. The solution has been to create an additional conda environment (pygssearch-env) that gets called from a subprocess. pygssearch is a command line tool and is called as such. This new envrionment is defined in a basic envrionment-pygssearch.yaml file
  • Preferential downloading from source is enabled for aqcuisition of both the scene and orbit files.
    • --scene_data_source can be one of ASF, AUS_COP_HUB, CDSE, or a space separated list of preferences
    • --orbit_data_source can be one of ASF, CDSE, or a space separated list of preferences
  • Tests have been introduced for downloading files from each available option. This includes logic that enables pixi to handle the additional conda environment for pygssearch.
  • Updates and renaming of docs
  • Minor changes to STAC metadata

Copy link
Collaborator

@geoscience-aman geoscience-aman left a comment

Choose a reason for hiding this comment

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

Thank you Alex, looking good.


## Workflows

For more information see [Workflows](docs/workflows/README.md)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Unrelated to this PR but can you please rename the current docs/workflows/README.MD to README.md for the link to work?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done nice pickup

Comment on lines +263 to +265
float(self.h5.search_value("centerFrequency")) / 1e9
) # GHz
self.item.properties["sarard:center_frequency_unit"] = "GHz"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think the CEOS metadata spec wants the center frequency in Hz?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Another conflict, the sar-extension wants it in GHz which is why I changed it. Given the xml will have us covered should I leave it?

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.

Thanks for adding the ability to use AusCopHub. My suggestions are minor and relate to making sure documentation is consistent with the API. The tests you've added look good to me.

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.

Thanks for making the changes!

@abradley60 abradley60 merged commit ec7885e into main Aug 18, 2025
2 checks passed
@caitlinadams caitlinadams deleted the updates/source_data_preferences branch August 18, 2025 05:56
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