Closed anayeaye closed 1 year ago
@anayeaye if we reprocess the images and upload with the same filename to the bucket, do we even need to re-inges into STAC? Wondering if we even need to remove and update stac.
@xhagrg True, if we're confident about the new COGs replacing them in place would work without updating the STAC records. If it is an easy/safe operation we wouldn't need to update STAC or delta-config. Maybe add some sort of thorough dashboard QA AC step instead?
Yes, when i get the reprocessing done, will send it in the veda-dashboard channel and ask for review/tests in the dashboard?
the cogs look okay in qgis.
@anayeaye updated the overviews and uploaded, still the same behavior.
it might be because of the min max zoom levels
At zooms > 0 for EPA emissions datasets such as Forest Fires (daily), we are getting 'Method returned empty array' errors from the tiler. The COGs are very small and it initially seemed like adding overviews and/or increasing the file size/decreasing pixel resolution could help but had no luck. For example the Abandoned Coal Mines COG was updated (see notes in this issue) but even with additional overviews and massive resizing we are not able to get tiles for higher zooms.
This seems like either a COG metadata problem or possibly some sort of caching in the tiler--or maybe even our map layer parameters? @vincentsarago do you see what might be going on here? Maybe we can tag up or look at the issue async sometime next week.
STAC Collection https://staging-stac.delta-backend.com/collections/EPA-daily-emissions_5_Forest_Fires
Sample unmodified COG copied to public bucket s3://covid-eo-data/20220923-epa-sample/EPA-daily-emissions_5_Forest_Fires_20120101.tif
Sample reprocessed COG copied to public bucket s3://covid-eo-data/20220923-epa-sample/EPA-annual-emissions_1B1a_Abandoned_Coal.tif
$ rio cogeo info EPA-daily-emissions_5_Forest_Fires_20120101.tif
...
Geo
Crs: EPSG:4326
Origin: (-129.99999694660497, 19.999999240339655)
Resolution: (0.09999999672558174, 0.0999999934417812)
BoundingBox: (-129.99999694660497, 54.99999694496308, -59.999999238697754, 19.999999240339655)
MinZoom: 3
MaxZoom: 4
...
BoundingBox: (-129.99999694660497, 54.99999694496308, -59.999999238697754, 19.999999240339655)
BoundingBox is really weird because it should be -129.99999694660497, 19.999999240339655, -59.999999238697754, 54.99999694496308,
This issue then is that rio-tiler will always return TileOutsideBounds
(en thus empty array) for this file because the geographic bounds is wrong!
Thanks @vincentsarago! I think this is the original processing script. I think we now have access to the raw data, so it sounds like we just need to try revisiting the nc to COG processing code check our bounds outputs until we get them right.
Will reprocess it and see if that works.
All of the EPA data is re-processed and currently loading fine.
Steps taken:
This issue is ready for QA: All of the COG assets for the EPA-* datasets in the EIS thematic area of the dashboard were reprocessed to solve a problem in which users could not zoom in on the map (the data became blurry at all zooms greater than 0). We need to make sure that all datasets are still viewable in the map and that users can zoom in without seeing a blurry map. cc: @gmarichalgxc @Catalina-Moller @mmaniceraGXC
Only the EPA - Other - Forest Fire (Daily) Dataset is blurry when the user zooms in.
Evidence:
All other EPA-* datasets in the EIS thematic area of the dashboard are still visible on the map and users can zoom in without seeing a blurry map.
cc: @anayeaye
They are not blurry anymore, but should the data really look like this - blobs with some kind of interpolation towards the center?
Do we know who the data provider was, then I can double-check with them?
The data was downloaded from https://www.epa.gov/ghgemissions/gridded-2012-methane-emissions#data. Original data is very coarse and we had to reprocess and upscale the images.
The EPA Datasets are rendering as expected. But we have found one possible issue that has a low priority, this is: NASA-IMPACT/veda-data#68
What
EPA assets cannot be zoomed > 0 in the dashboard. The fix is to recreate the COG assets with appropriate overviews.
EPA COG info snippet
LIS TWS COG info snippet for comparison w/overviews
AC