Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Add ECCO ocean state estimate #217

Closed
4 of 7 tasks
j08lue opened this issue Nov 1, 2022 · 10 comments
Closed
4 of 7 tasks

Add ECCO ocean state estimate #217

j08lue opened this issue Nov 1, 2022 · 10 comments
Assignees
Labels

Comments

@j08lue
Copy link
Contributor

j08lue commented Nov 1, 2022

  • Identify data location s3://veda-data-store-staging/EIS/COG/SSH_2017_minus_1992.cog.tif

  • Number of items: 1

  • Identify the point of contact and ensure someone is providing them updates: Denis, @j08lue to follow up

  • Verify data is valid COG

  • Gather STAC collection metadata

    • temporal interval:
    • license:
    • id:
    • title: ECCO sea-surface height change from 1992 to 2017
    • description:
      Gridded global sea-surface height change from 1992 to 2017 from the Estimating the Circulation and Climate of the Ocean (ECCO) ocean state estimate. The dataset was calculated as the difference between the annual means over 2017 and 1992, from the 0.5 degree, gridded monthly mean data product available on PO.DAAC (https://podaac.jpl.nasa.gov/dataset/ECCO_L4_SSH_05DEG_MONTHLY_V4R4).
    • whether it is periodic or not on the dashboard: periodic: false
    • the dashboard time density:
  • Review and follow https://github.com/NASA-IMPACT/cloud-optimized-data-pipelines/blob/main/OPERATING.md

  • Once the PR is approved, we can merge and publish those datasets to the Staging API

Resources about metadata

@j08lue j08lue added the dataset label Nov 1, 2022
@anayeaye
Copy link
Contributor

anayeaye commented Nov 1, 2022

STAC metadata notes

  • Number of items: 1
  • dashboard:is_periodic = false, the dashboard will not need to construct a series of periodic dates for the time picker
  • dashboard:time_density = null, there are not daily, monthly, etc items

Asset notes

@dfelikson
Copy link

Sorry about this! I'm not sure what happened but I provided a bad version of the tif. I've replaced the COG with a version that does contain data: s3://veda-data-store-staging/EIS/COG/SSH_2017_minus_1992.cog.tif

@j08lue
Copy link
Contributor Author

j08lue commented Nov 2, 2022

Great, looks good now:

image

@ividito ividito self-assigned this Nov 2, 2022
@anayeaye
Copy link
Contributor

anayeaye commented Nov 2, 2022

companion delta-config issue NASA-IMPACT/veda-config#128

@j08lue
Copy link
Contributor Author

j08lue commented Nov 11, 2022

Wait a sec. There might be an issue with this COG (or some mosaic.json?), @dfelikson. When I preview it in the COG viewer (/cog/viewer), it does not zoom:

image

And I can see from the settings for this dataset (/cog/validate) that it uses

{"MinZoom":0,"MaxZoom":1}

@vincentsarago, could I ask you to help explain where MinZoom and MaxZoom come from? Is that setting stored in the COG header or does it come from some mosaic.json? I can see this is a delicate piece of information... developmentseed/titiler#291

@vincentsarago
Copy link
Collaborator

@j08lue in /cog/validate the zoom informations are derived from the size of the file (resolution) once reprojected to the WebMercator TMS. maxzoom=1 could mean that either your file is small or there is an issue with it's bounds (or there is a bug in rio-tiler 😓)

can you share the result of /cog/info for the file?

For the mosaic, the min/max zoom are usually equal to the zoom available for the COGs

@j08lue
Copy link
Contributor Author

j08lue commented Nov 11, 2022

https://staging-raster.delta-backend.com/cog/viewer?url=s3://veda-data-store-staging/EIS/COG/SSH_2017_minus_1992.cog.tif

https://staging-raster.delta-backend.com/cog/info?url=s3://veda-data-store-staging/EIS/COG/SSH_2017_minus_1992.cog.tif

{"bounds":[-180.0,-90.0,180.0,90.0],"minzoom":0,"maxzoom":1,"band_metadata":[["1",{}]],"band_descriptions":[["1",""]],"dtype":"float32","nodata_type":"Nodata","colorinterp":["gray"],"width":720,"nodata_value":-9999.0,"height":360,"overviews":[2],"driver":"GTiff","count":1}

@vincentsarago
Copy link
Collaborator

"height":360

It's small dataset covering a huge area so the resolution is big and this the max Zoom is low

@j08lue
Copy link
Contributor Author

j08lue commented Nov 11, 2022

Oh, my bad. Sorry for the fuss! I have rarely worked with data that coarse lately, therefore I did not even consider looking at the native resolution... 🤦

All as it should be, then, false alarm.

@Catalina-Moller
Copy link

The Dataset works as expected!

But we have seen that the date model shown in the dataset is the old version, the new version should be shown.

Evidence:

Image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

6 participants