Skip to content

add schema for landCover, base theme#118

Merged
Victor Schappert (vcschapp) merged 20 commits intodevfrom
landcover
May 8, 2024
Merged

add schema for landCover, base theme#118
Victor Schappert (vcschapp) merged 20 commits intodevfrom
landcover

Conversation

@jonahadkins
Copy link
Copy Markdown
Contributor

@jonahadkins Jonah Adkins (jonahadkins) commented Feb 6, 2024

Proposal

Add multiple resolutions of global land cover data to base theme in Overture (sometime in 2024). Examples below from Meta's production basemaps.

Screenshot 2024-02-06 at 9 17 39 AM

Low Resolution, minZoom=0, maxZoom=4

Screenshot 2024-02-06 at 9 18 12 AM

Medium Resolution, minZoom=4, maxZoom=9

Screenshot 2024-02-06 at 9 20 16 AM

High Resolution, minZoom=9, maxZoom=24

Background

The Display Maps team at Meta recently completed a project to integrate global land cover data into it's basemaps. Landcover data provides an opportunity o display continuous, gapless representation of the Earth's natural surfaces. We derived these products from the 2020 European Space Agency (ESA) raster land cover product. Our process was generally: (1) download the image tiles, (2) process them, (3) vectorize them, and finally (4) post-process them to a usable format for vector tiling. We publicly released these with our Daylight product last year (Blog).

For Discussion

(1) IMO this is a huge opportunity for Overture to provide a massive dataset, critical to every global basemap used in a multitude of mapping applications. This would be a first-of-its kind public dataset, it's hard to develop for day-to-day mappers, and is a huge technical lift at global scale.

Examples of land cover in production across several providers.
compare

(2) Providing multiple resolutions meets the needs of map developers and cartographers who would use this data in their workflows. We could provide only the highest resolution, but users would need to develop complex processes to generalize and simplify the data. Providing only medium or low resolution would fall short of expectations for this type of dataset.

Providing tile zoom recommendations is probably a better way to define resolution than a specific reference scale like 1:1,000,000 or text like 'medium'.

(3) This is typically a "one and done" type dataset. This type of data is generally only needs refreshing every 5 to 10 years, and adding metadata such as year and source helps define that.

(4) While this type of data can be found in OpenStreetMap, this product provides continuous coverage across the globe.

@jonahadkins Jonah Adkins (jonahadkins) changed the title add schema for landCover, base theme [WIP] add schema for landCover, base theme Feb 6, 2024
@vcschapp
Copy link
Copy Markdown
Collaborator

This sounds great!

I feel like the right place to initially have this discussion is within the base theme Task Force. Once they're aligned that:

  1. this is a good thing to do; and
  2. we can generate and provide the data once the schema is merged;

then we should push forward with it...

@jonahadkins
Copy link
Copy Markdown
Contributor Author

I feel like the right place to initially have this discussion is within the base theme Task Force.

thanks Victor Schappert (@vcschapp) - this PR is the result of discussion in that task force last week

Copy link
Copy Markdown
Contributor

@BoZivk BoZivk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be referenced within schema/schema.yaml?

BoZivk
BoZivk previously approved these changes Feb 21, 2024
@jonahadkins
Copy link
Copy Markdown
Contributor Author

BoZivk
BoZivk previously approved these changes Mar 29, 2024
@jenningsanderson Jennings Anderson (jenningsanderson) changed the base branch from main to dev April 11, 2024 15:29
@jenningsanderson Jennings Anderson (jenningsanderson) dismissed stale reviews from BoZivk and themself April 11, 2024 15:29

The base branch was changed.

@jonahadkins Jonah Adkins (jonahadkins) changed the title [WIP] add schema for landCover, base theme add schema for landCover, base theme Apr 11, 2024
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a couple of examples and counter-examples specific to the land_cover feature type?

adding to defs
fixing build error?
fix defs?
remove propers, use container
add land cover to featureType
@jenningsanderson
Copy link
Copy Markdown
Collaborator

Jonah Adkins (@jonahadkins) Wasn't there a version of this where min-max zoom level was expressed instead as a scale?

@jonahadkins
Copy link
Copy Markdown
Contributor Author

Jonah Adkins (@jonahadkins) Wasn't there a version of this where min-max zoom level was expressed instead as a scale?

originally, but we thought it was more confusing

Copy link
Copy Markdown
Collaborator

@jwass Jacob Wasserman (jwass) left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking forward to getting this out there!

sentence case descriptions
@jwass
Copy link
Copy Markdown
Collaborator

PR for data license approval: https://github.com/OvertureMaps/tf-data-platform/pull/253

@vcschapp
Copy link
Copy Markdown
Collaborator

In schema meeting 2024-05-01, Jacob Wasserman (@jwass) indicated including ESA WorldCover for use as landcover is OK by legal. So this is not a blocker for merging.

remove links
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.

6 participants