search

opengeospatial / geopackage-tiled-gridded-coverage

A GeoPackage extension for tiled, gridded coverage data
Apache License 2.0
5 stars 8 forks source link

mismatch for `tile_matrix_set_id` #8

Closed bradh closed 9 years ago

bradh commented 9 years ago

The gpkg_2D_gridded_coverage_ancillary table definition has a column tile_matrix_set_id, which is supposed to be an integer that is a foreign key to id in gpkg_tile_matrix_set.

Unfortunately there is no id in gpkg_tile_matrix_set : http://www.geopackage.org/spec/#tile_matrix_set_data_table_definition

Suggest this be a FK to the table_name in gpkg_tile_matrix_set. It could also be possible to use the rowid if we are confident that no-one creates gpkg_tile_matrix_set using the WITHOUT ROWID optimisation (https://www.sqlite.org/withoutrowid.html).

jyutzler commented 9 years ago

https://github.com/opengeospatial/geopackage-elevation/blob/master/spec/1_tiled_gridded_elevation_data.adoc#gpkg_2D_gridded_coverage_ancillary_table

jyutzler commented 9 years ago

In today's SWG we agreed to change id to table_name.