eurodatacube / eodash-catalog

MIT License
1 stars 0 forks source link

Water quality maps and sea surface temperature new AOI missing #135

Closed AlessandroScremin closed 3 months ago

AlessandroScremin commented 3 months ago

@lubojr @santilland CC: @dmoglioni

I see htat the new AOIs (Portugal and Hamburg) for water quality maps from CNR are wrongly exposed both in eodashboard and Race

you see that filtering for Total suspended matter maps of ESA

There are only 4 AOI (out of 6)

image

but when I zoom in I see the new aoi but not POI avaialbe

image

What is needed to add the two new AOI as POI?

lubojr commented 3 months ago

@AlessandroScremin please update the configuration YAMLs with the new desired pois in a new branch and create a PR. You can then review the state of the catalog generated if you add following text to the PR description: N3a2_chl_concentration_tri_esa N3a2_total_suspended_matter_tri_esa

on these places https://github.com/eurodatacube/eodash-catalog/blob/main/collections/N3a2_total_suspended_matter_tri_esa.yaml#L19 https://github.com/eurodatacube/eodash-catalog/blob/main/collections/N3a2_chl_concentration_tri_esa.yaml#L19

The catalog will be deployed on https://eodashcatalog.eox.at/branchname/trilateral/catalog.json and state of the client can be tested on https://eodash-trilateral-staging.eox.at/?catalog=branchname

Replace branchname with your newly created branch in this repo

AlessandroScremin commented 3 months ago

@lubojr changed the yamls file (3 of them) and created a pull request.

lubojr commented 3 months ago

@AlessandroScremin The build failed to run through https://github.com/eurodatacube/eodash-catalog/actions/runs/8967270206/job/24624466859 the PR description collection names can not have the .yaml suffix and collections should be delimited by space , not newline. I have updated the PR description and rerun the pipeline now. On first glance it looks fine, so I will approve the PR and feel free to merge it whenever you want to. (I think you should have the permissions).

lubojr commented 3 months ago

Will get fixed when https://github.com/eurodatacube/eodash-catalog/pull/137 is merged. Closing the issue.