search

ua-snap / rasdaman-ingest

Collection of ingredients/configurations + docs for ingesting data into Rasdaman
MIT License
3 stars 0 forks source link

Updates Rasdaman coverage ingest files to match the required changes in newer versions of Rasdaman #76

Closed BobTorgerson closed 5 months ago

BobTorgerson commented 6 months ago

This PR modifies the Rasdaman ingest files for all of the coverages to have a non-blank title. When there is a blank title in the newest version of Rasdaman, it completes the ingest, but the coverage is unusable due to the XML document being in an unexpected state.

In addition, the API was scraping the metadata for fields that originally resided in the "global" tag in the coverages "metadata" tag. However, with the new change to Rasdaman, that field is no longer including undefined tags so they were simply being thrown away when left in the global tag. After some research and trial-and-error, I discovered that by putting these into the "local" tag of the metadata, it was again making those metadata fields available for scraping by our data API.

These changes will be required for any new coverages we create to work with Rasdaman 10+.

Please let me know if you have any questions!

charparr commented 5 months ago

Hey I'm inclined to go ahead approve this! I think this was the PR that was paired with the HISA prefect flow? That flow I did test and execute specifically. @BobTorgerson if there is something specific I need to test here please let me know! I don't want to block the rasdaman branch settling.