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

chore: adding placeholder yaml files for sciencehub results #186

Merged
merged 20 commits into from
Sep 26, 2024
Merged

Conversation

santilland
Copy link
Collaborator

@santilland santilland commented Aug 13, 2024

SHC1_era5_u_winds SHC2_era5_v_winds SHC3_tropomi_cube SHC4_propagated_uncertainty

@santilland
Copy link
Collaborator Author

Datasets available here:
https://xcube.deepesdl.brockmann-consult.de/api/datasets
There is an issue with identifiers having a "/" and breaking references in catalog, etc, waiting for resolution

@aapopescu
Copy link
Contributor

Hi, @FedericoRondoni, the metadata for the "SHC4_propagated_uncertainty" is not very clear.

from the filename it seems the dataset is related to Sentinel-3 (C3S-L3_AEROSOL-AER_PRODUCTS-SLSTR-SENTINEL3B-SDV-MONTHLY-v2.10.zarr) but this is not visible in the metadata.

Can you please double check and correct?

@FedericoRondoni
Copy link
Contributor

Hi, @FedericoRondoni, the metadata for the "SHC4_propagated_uncertainty" is not very clear.

from the filename it seems the dataset is related to Sentinel-3 (C3S-L3_AEROSOL-AER_PRODUCTS-SLSTR-SENTINEL3B-SDV-MONTHLY-v2.10.zarr) but this is not visible in the metadata.

Can you please double check and correct?

Hi @aapopescu, the yaml file for SHC4 is still to be reviewed because it was set to low priority. Thus, I had left it for when I was coming back. I will work on it and upload the correct version

@FedericoRondoni
Copy link
Contributor

Hi @santilland I think there is an error inside the SHC4 data location. Can you check?

@santilland
Copy link
Collaborator Author

@FedericoRondoni looking at the available collections it seems the collection id does not end with .nc but with .zarr, i can see following id responded by the api:
scienhub-netcdf-cubes~202001-C3S-L3_AEROSOL-AER_PRODUCTS-SLSTR-SENTINEL3B-SDV-MONTHLY-v2.10.zarr

@aapopescu
Copy link
Contributor

@santilland , Can you please provide the testing URL? thanks!

@santilland
Copy link
Collaborator Author

santilland commented Sep 4, 2024

@aapopescu @FedericoRondoni you can see the available collections under:
https://xcube.deepesdl.brockmann-consult.de/api/ogc/collections/
Is this what you meant?

**edit: just came to my mind, maybe you were thinking about the dashboard?
https://eodash-trilateral-staging.eox.at/?catalog=sciencehub

@FedericoRondoni
Copy link
Contributor

@FedericoRondoni looking at the available collections it seems the collection id does not end with .nc but with .zarr, i can see following id responded by the api: scienhub-netcdf-cubes~202001-C3S-L3_AEROSOL-AER_PRODUCTS-SLSTR-SENTINEL3B-SDV-MONTHLY-v2.10.zarr

Hi @santilland thanks a lot! This solves the issue

@lubojr
Copy link
Member

lubojr commented Sep 10, 2024

@FedericoRondoni We've fixed the wrong Query config and the SHC4_propagated_uncertainty is now built properly.

@lubojr
Copy link
Member

lubojr commented Sep 11, 2024

@lubojr
Copy link
Member

lubojr commented Sep 13, 2024

@aapopescu we added references to the legends for the 4 datasets. This PR can be merged from our point of view.

@lubojr lubojr requested a review from aapopescu September 13, 2024 14:51
@lubojr
Copy link
Member

lubojr commented Sep 24, 2024

Confirmed on meeting that is good to merge once assets are finalized and merged. Latest friday midday.

@santilland
Copy link
Collaborator Author

Endpoint was converted to version 1.7.0 and provides wrong stac response, waiting for fix

@lubojr lubojr merged commit e93f3d6 into main Sep 26, 2024
1 check passed
@lubojr lubojr deleted the sciencehub branch September 26, 2024 20:31
@lubojr
Copy link
Member

lubojr commented Sep 26, 2024

Endpoint was reverted to 1.6.0 and pipeline is unblocked. Merging

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.

4 participants