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

Add wktLiteral @type for geojson:geometry #37

Closed
wants to merge 1 commit into from

Conversation

sgillies
Copy link
Contributor

Towards closing #32.

This helps RDF users get usable RDF by coercion of GeoJSON geometry to WKT.

It does nothing for RDF -> GeoJSON, but I don't think this lack has to be a blocker for a first release of GeoJSON-LD on the web.

Definition of a new container types specific to GeoJSON geometry objects and N-D arrays could assist with RDF -> GeoJSON conversions as well as serialization of geometries to pure RDF, but let's defer work on those.

Reviews welcome!

@sgillies sgillies added this to the GeoJSON-LD 1.0 milestone Oct 29, 2015
@pietercolpaert
Copy link

I'd vote against merging this as:

  • The semantics are not right: what's given in the JSONLD is not a WKT string
  • I'm still not convinced about fixing this with vanilla JSONLD parsers, as @dinizime puts it, in Coordinates to RDF doesn't work #32. I'm more in favour of fixing this by introducing something like @values in JSON-LD, which describes the values of multidimensional arrays.

@pietercolpaert
Copy link

I suggest waiting for json-ld/json-ld.org#397. If this proposal gets shot by the JSON-LD community, you can go ahead and merge this.

@sgillies
Copy link
Contributor Author

@pietercolpaert thank you very much! I'll follow that issue.

@akuckartz
Copy link

@sgillies Thanks for trying to resolve these difficult and often controversial issues!

@pietercolpaert
Copy link

I don't understand how the @type is a WKT Literal, as it isn't: it's a JSON array. No?

@sgillies
Copy link
Contributor Author

sgillies commented Nov 3, 2015

@pietercolpaert my intent is that @type would trigger coercion to WKT as in http://www.w3.org/TR/json-ld/#type-coercion. Am I misreading that section?

@pietercolpaert
Copy link

@sgillies No, you aren't. My bad!

@sgillies
Copy link
Contributor Author

sgillies commented Nov 3, 2015

@pietercolpaert I'm glad to get second opinions. Sometimes I feel lost without a compass in JSON-LD!

@sgillies sgillies modified the milestone: GeoJSON-LD 1.0 Nov 18, 2015
@sgillies
Copy link
Contributor Author

@pietercolpaert it looks like we should merge and continue without creation of a new media type. Agreed?

Related: I've read the discussion in w3c/activitystreams#52 and have a better understanding of where many commenters here are coming from.

@sgillies
Copy link
Contributor Author

Closing.

@sgillies sgillies closed this May 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants