/geopackage

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

Sweep requirements, ATS tests, and executable tests for alignment #308

Closed
jyutzler opened this Issue Mar 3, 2017 · 2 comments

Comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  1.2-comment period
1 participant
@jyutzler
@jyutzler
Contributor

jyutzler commented Mar 3, 2017

We need to make sure everything is aligned.

@jyutzler jyutzler added this to the 1.2-comment period milestone Mar 3, 2017

@jyutzler jyutzler referenced this issue Mar 3, 2017

Merged

I308 #310

@jyutzler

This comment has been minimized.

Sign in to view
Contributor

jyutzler commented Mar 3, 2017 •
edited

/base/core/spatial_ref_sys/data_values_required is unfortunately positioned. There should be individual requirements under the tiles and features parts of the standard. Since attributes are inherently non-spatial, their SRSs can be null. This makes the ATS invalid as written. Fortunately or not, the ETS was not using the test prescribed in the ATS. I will update the ATS to match the ETS.
@jyutzler

This comment has been minimized.

Sign in to view
Contributor

jyutzler commented Mar 13, 2017

Today the SWG approved these changes to the ATS

@jyutzler jyutzler closed this Mar 13, 2017

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