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

Notes and Open Annotation #330

Closed
gkellogg opened this issue Mar 10, 2015 · 15 comments
Closed

Notes and Open Annotation #330

gkellogg opened this issue Mar 10, 2015 · 15 comments

Comments

@gkellogg
Copy link
Member

Issue #288 has a comment suggesting that notes would be clearer if it included Open Annotation content, such as the following:

"notes": {
    "@type": "oa:Annotation",
    "oa:hasTarget": "http://example.org/tree-ops-ext",
    "oa:hasBody": {
      "@type": ["cnt:ContentAsText", "dctypes:Text"],
      "cnt:chars": "This is a very interesting comment about the table; it's a table!",
      "dc:format": {"@value": "text/plain"}
    }
}

Note that none of the prefixes "oa", "cnt" or ""dctypes" are in our context, as they're not in the RDFa initial context.

@6a6d74
Copy link
Contributor

6a6d74 commented Mar 10, 2015

Gotcha. Am in the process of adding notes example to csv2rdf and csv2json.

I spotted the initial context issue.

@iherman
Copy link
Member

iherman commented Mar 10, 2015

On 10 Mar 2015, at 13:34 , Gregg Kellogg [email protected] wrote:

Issue #288 has a comment suggesting that notes would be clearer if it included Open Annotation content, such as the following:

"notes": {
"@type": "oa:Annotation",
"oa:hasTarget": "http://example.org/tree-ops-ext",
"oa:hasBody": {
"@type": ["cnt:ContentAsText", "dctypes:Text"],
"cnt:chars": "This is a very interesting comment about the table; it's a table!",
"dc:format": {"@value": "text/plain"}
}
}

Note that none of the prefixes "oa", "cnt" or ""dctypes" are in our context, as they're not in the RDFa initial context.

Indeed:-(

'oa', per our rules, eventually will (as soon as OA becomes a CR) and it is already flagged as one. 'dctypes'... well, it did not make it back then, for whatever reasons; I do not see any problems adding these two to our initial context now (although it is a bit of a problem that we have to add our own but, anyway).

'cnt' is a different animal. The corresponding spec[1] does not seem to go anywhere, so the now really authoritative version of oa[2] has dropped its usage. Ie, we should not do that either. B.t.w., it would be better to look at [2] for examples that we would reuse; one advantage is that the examples are (also) in JSON-LD...

Ivan

[1] http://www.w3.org/TR/Content-in-RDF10/
[2] http://www.w3.org/TR/annotation-model/


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

We should also add a prefix rel: http://www.iana.org/assignments/link-relations/, if that's what we've decided on.

@iherman
Copy link
Member

iherman commented Mar 10, 2015

That is definitely pending, see issue #297 !

@gkellogg
Copy link
Member Author

This might be a better example, then:

 "notes": {
     "@type": "oa:Annotation",
     "oa:hasTarget": "http://example.org/tree-ops-ext",
     "oa:hasBody": {
       "@type": "oa:EmbeddedContent",
       "rdf:value": "This is a very interesting comment about the table; it's a table!",
       "dc:format": {"@value": "text/plain"}
     }
 }

Note that we could use dc:language, however we set @language, so this is implicit in the embedded value. I think dc:language might be more important for describing the language used in external media.

@iherman
Copy link
Member

iherman commented Mar 10, 2015

Yes, that looks good!

On 10 Mar 2015, at 15:32 , Gregg Kellogg [email protected] wrote:

This might be a better example, then:

"notes": {
"@type": "oa:Annotation",
"oa:hasTarget": "http://example.org/tree-ops-ext",
"oa:hasBody": {
"@type": "oa:EmbeddedContent",
"rdf:value": "This is a very interesting comment about the table; it's a table!",
"dc:format": {"@value": "text/plain"}
}
}

Note that we could use dc:language, however we set @language, so this is implicit in the embedded value. I think dc:language might be more important for describing the language used in external media.


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 added a commit that referenced this issue Mar 10, 2015
@gkellogg gkellogg mentioned this issue Mar 10, 2015
@gkellogg
Copy link
Member Author

@6a6d74 I may have beaten you to it (notes examples) in #333; sorry if there's an overlap.

@JeniT
Copy link

JeniT commented Mar 10, 2015

Is this issue actually about a proposed technical change or purely editorial (ie about an example to use)? If the former, what's the proposed technical change?

@iherman
Copy link
Member

iherman commented Mar 10, 2015

It is editorial, except that some prefixes are added to the default context.

Ivan


Ivan Herman
Tel:+31 641044153
http://www.ivan-herman.net

(Written on mobile, sorry for brevity and misspellings...)

On 10 Mar 2015, at 16:39, Jeni Tennison [email protected] wrote:

Is this issue actually about a proposed technical change or purely editorial (ie about an example to use)? If the former, what's the proposed technical change?


Reply to this email directly or view it on GitHub.

@gkellogg
Copy link
Member Author

Without objection, I'll simply merge.

@iherman
Copy link
Member

iherman commented Mar 10, 2015

go ahead


Ivan Herman
+31 641044153

(Written on my mobile. Excuses for brevity and frequent misspellings...)

On 10 Mar 2015, at 17:32, Gregg Kellogg [email protected] wrote:

Without objection, I'll simply merge.


Reply to this email directly or view it on GitHub.

@6a6d74
Copy link
Contributor

6a6d74 commented Mar 12, 2015

@gkellogg - I've added a bit more stuff to the csv2rdf and csv2json specs and merged; see PR #347

@6a6d74
Copy link
Contributor

6a6d74 commented Mar 12, 2015

can we close this issue now? I think that all the actions are done ...

@iherman
Copy link
Member

iherman commented Mar 12, 2015

having looked at the files I think yes

@6a6d74
Copy link
Contributor

6a6d74 commented Mar 12, 2015

thanks @iherman ... closing the issue.

@6a6d74 6a6d74 closed this as completed Mar 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants