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

JSON Common properties require removing JSON-LDisms #344

Closed
gkellogg opened this issue Mar 12, 2015 · 8 comments · Fixed by #372
Closed

JSON Common properties require removing JSON-LDisms #344

gkellogg opened this issue Mar 12, 2015 · 8 comments · Fixed by #372

Comments

@gkellogg
Copy link
Member

The algorithm says to simply copy common properties and notes in place, however these have been normalized, and now simple IRI references and values are in expanded form. A simple de-normalize algorithm is necessary to do the following:

  • Turn values of the form {"@value": "something"} into just "something", also ignoring @language or @type included in the value object.
  • Turn nodes of the form {"@id": "http://example.com/someplace"} into simply "http://example.com/someplace".
@gkellogg
Copy link
Member Author

As with the RDF version, values of @id should also be expanded relative to the document base.

@6a6d74
Copy link
Contributor

6a6d74 commented Mar 12, 2015

Furthermore, do you think that name-value pairs specifying @type should be included in the (plain old) JSON? e.g.

    "notes": {
      "@type": "oa:Annotation",
      [... etc. ...]
      }

(snippet from tree-ops-ext-standard.json)

@iherman
Copy link
Member

iherman commented Mar 12, 2015

On 12 Mar 2015, at 06:38 , Jeremy Tandy notifications@github.com wrote:

Furthermore, do you think that name-value pairs specifying @type should be included in the (plain old) JSON? e.g.

"notes": {
  "@type": "oa:Annotation",
  [... etc. ...]
  }

I think it requires more work for the implementation to take it out than to just leave it there (although it is not very meaningful for non-LD JSON...)

Ivan


Reply to this email directly or view it on GitHub.


Ivan Herman, W3C
Digital Publishing Activity Lead
Home: http://www.w3.org/People/Ivan/
mobile: +31-641044153
ORCID ID: http://orcid.org/0000-0003-0782-2704

@gkellogg
Copy link
Member Author

I think it's useful information that doesn't make the data less easy to use, so it should stay in. Note that we use @id in the output, giving the impression that the output is JSON-LD, which it's not. But, if someone came up with a co txt, they could reasonably treat it as if it were, which isn't a bad thing.

@6a6d74
Copy link
Contributor

6a6d74 commented Mar 12, 2015

@iherman, @gkellogg ... good points. Just thought I'd ask

@6a6d74
Copy link
Contributor

6a6d74 commented Mar 13, 2015

Obviously this applies to processing notes too. [UPDATE: already says this at top - sorry]

@6a6d74
Copy link
Contributor

6a6d74 commented Mar 18, 2015

Can we confirm that @gkellogg's proposal (at the beginning of this ISSUE) is sufficient? If so I can add this to the doc.

@iherman
Copy link
Member

iherman commented Mar 18, 2015

Yes

Ivan

On 18 Mar 2015, at 15:52 , Jeremy Tandy notifications@github.com wrote:

Can e confirm that @gkellogg's proposal (at the beginning of this ISSUE) is sufficient? If so I can add this to the doc.


Reply to this email directly or view it on GitHub.


Ivan Herman, W3C
Digital Publishing Activity Lead
Home: http://www.w3.org/People/Ivan/
mobile: +31-641044153
ORCID ID: http://orcid.org/0000-0003-0782-2704

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

Successfully merging a pull request may close this issue.

4 participants