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

Inconsistency in Figure 6. GeoPackage Minimal Tables for Tiles #117

Closed
xeonx opened this Issue Jul 16, 2015 · 3 comments

Comments

Projects
None yet
2 participants
@xeonx

xeonx commented Jul 16, 2015

In my understanding as the primary key of gpkg_tile_matrix_set is the same as the one of gpkg_contents (table_name) the association in the diagram should be 1..1 instead of 1..0-*

An other possibility is to have srs_id column included in the gpkg_tile_matrix_set primary key, allowing to store multiple projections of the same content. But in this case the srs_id column shall also be added in the gpkg_tile_matrix table and primary key.

@jyutzler jyutzler added this to the tiles-corrigendum milestone Jul 21, 2015

@jyutzler jyutzler self-assigned this Jul 21, 2015

@jyutzler

This comment has been minimized.

Contributor

jyutzler commented Jul 24, 2015

I talked this over with @pdaisey today and we determined that the associations in question should be 1..0-1. The reason for the '0-' is because a gpkg_contents entry doesn't necessarily correspond to a gpkg_tile_matrix_set entry (it might be features instead). We suspect that the '*' multiplicity is an oversight that was not corrected when we prohibited multiple geometry columns per feature table.

It will take me longer to acquire and install the software used to make this diagram than it will to make the actual changes.

jyutzler added a commit that referenced this issue Aug 4, 2015

jyutzler added a commit that referenced this issue Aug 4, 2015

@jyutzler

This comment has been minimized.

Contributor

jyutzler commented Aug 4, 2015

Approved in SWG today. See PR #126

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment