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

Proposal for geometry.precision #37

Open
docuracy opened this issue Feb 24, 2022 · 5 comments
Open

Proposal for geometry.precision #37

docuracy opened this issue Feb 24, 2022 · 5 comments
Labels
enhancement New feature or request JSON Schema

Comments

@docuracy
Copy link
Collaborator

docuracy commented Feb 24, 2022

There is almost always some approximation in the expression of Point coordinates, and it would be useful to be able to indicate the degree of approximation. For example, archaeological find sites may need to be obfuscated and indicated by a grid square; locations of other features might be recorded only as within a town, county, or country.

PROPOSAL: Introduce an optional geometry.precision property, which itself has one of the following properties as outlined in this draft JSON schema:

  • tolerance
  • bbox
  • ccodes
  • citations
@docuracy docuracy added enhancement New feature or request JSON Schema labels Feb 24, 2022
@rybesh
Copy link
Contributor

rybesh commented Feb 26, 2022

Elsewhere (see e.g. jazzband/geojson#135) precision is used to mean the number of decimal places in decimal coordinates, so another term might be preferable here.

I also wonder whether this is something that needs to be in LPF rather than something that applications that need it could layer on top of LPF. I imagine that specific applications may have all kinds of ways that they want to deal with precision that might be difficult to anticipate ahead of time.

@docuracy
Copy link
Collaborator Author

Agreed. We're not really looking at precision but rather at granularity. Would that do?

In my experience granularity is such a common question when dealing with geodata that I think it ought to be (an optional) part of the core LP specification. That being the case, I think we should at least try to anticipate how it might be dealt with, and revise the specification as and when other use cases become apparent.

@rybesh
Copy link
Contributor

rybesh commented Mar 7, 2022

Yes, I like granularity. A lot of tools use “zoom levels” for this so we might try to find a way of expressing this that is compatible with that, i.e. maybe “zoom level” is one of the ways of expressing granularity.

@rayrrr
Copy link

rayrrr commented May 6, 2022

Thanks for this suggestion @docuracy; also I agree that granularity is a good name for this concept. However, a larger conversation has to be had about extending the scope of this package. This package's mandate is to tightly track the GeoJSON standard, IETF RFC 7946. As such, it would be preferable to create a new Python package specifically for supporting the Linked Places format/schema instead, as it is not a part of the GeoJSON standard. Thoughts?

@rybesh
Copy link
Contributor

rybesh commented May 6, 2022

@rayrrr This issue is not about a Python package; it's about the schema for Linked Places Format. I think I might have created confusion when I mentioned the jazzband/geojson issue above—I just did that to raise the point about naming.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request JSON Schema
Projects
None yet
Development

No branches or pull requests

3 participants