Skip to content
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

Reword Requirement #92 #362

Closed
jyutzler opened this issue May 8, 2017 · 1 comment
Closed

Reword Requirement #92 #362

jyutzler opened this issue May 8, 2017 · 1 comment

Comments

@jyutzler
Copy link
Contributor

@jyutzler jyutzler commented May 8, 2017

Requirement 92 is currently unclear.

(extends GPKG-36 and GPKG-37) A GeoPackage that contains a tile pyramid user data table that contains tile data MAY store tile_data in MIME type image/x-webp [22]. The MIME type of values of the tile_data column in tile pyramid user data tables SHALL be image/x-webp.

Does this mean that you can have PNG or JPG tiles or not? The first sentence suggests you can, the second suggests you can't. I believe that the first sentence matches the spirit and intent of the requirement. A better wording might be:

(extends GPKG-36 and GPKG-37) A GeoPackage that contains a tile pyramid user data table that contains tile data MAY store tile_data in the WebP format [22]. Files complying with the WebP format SHALL have the MIME type of image/x-webp.

Ref: opengeospatial/ets-gpkg12#11
h/t @rouault

@jyutzler jyutzler added this to the 1.2-comment period milestone May 8, 2017
@rouault
Copy link
Contributor

@rouault rouault commented May 8, 2017

Files complying with the WebP format SHALL have the MIME type of image/x-webp.

Tiles rather than files ? But I don't understand what this sentence brings. Saying that tile_data may be in the WebP format is that same than saying that the MIME type is image/x-webp.

jyutzler added a commit to jyutzler/geopackage that referenced this issue Jun 2, 2017
jyutzler added a commit that referenced this issue Jun 6, 2017
#362 clarifying whether you can mix or match JPG/PNG/WebP
@jyutzler jyutzler closed this Jun 6, 2017
kwrobot pushed a commit to aashish24/gdal-svn that referenced this issue Jun 6, 2017
rouault added a commit to OSGeo/gdal that referenced this issue Jun 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.