{"action":"created","issue":{"url":"https://api.github.com/repos/w3c/csvw/issues/469","labels_url":"https://api.github.com/repos/w3c/csvw/issues/469/labels{/name}","comments_url":"https://api.github.com/repos/w3c/csvw/issues/469/comments","events_url":"https://api.github.com/repos/w3c/csvw/issues/469/events","html_url":"https://github.com/w3c/csvw/pull/469","id":67213251,"number":469,"title":"Issue 463: Change the `value` term in the model document to `annotation value`.","user":{"login":"gkellogg","id":46296,"avatar_url":"https://avatars.githubusercontent.com/u/46296?v=3","gravatar_id":"","url":"https://api.github.com/users/gkellogg","html_url":"https://github.com/gkellogg","followers_url":"https://api.github.com/users/gkellogg/followers","following_url":"https://api.github.com/users/gkellogg/following{/other_user}","gists_url":"https://api.github.com/users/gkellogg/gists{/gist_id}","starred_url":"https://api.github.com/users/gkellogg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gkellogg/subscriptions","organizations_url":"https://api.github.com/users/gkellogg/orgs","repos_url":"https://api.github.com/users/gkellogg/repos","events_url":"https://api.github.com/users/gkellogg/events{/privacy}","received_events_url":"https://api.github.com/users/gkellogg/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"JeniT","id":395050,"avatar_url":"https://avatars.githubusercontent.com/u/395050?v=3","gravatar_id":"","url":"https://api.github.com/users/JeniT","html_url":"https://github.com/JeniT","followers_url":"https://api.github.com/users/JeniT/followers","following_url":"https://api.github.com/users/JeniT/following{/other_user}","gists_url":"https://api.github.com/users/JeniT/gists{/gist_id}","starred_url":"https://api.github.com/users/JeniT/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JeniT/subscriptions","organizations_url":"https://api.github.com/users/JeniT/orgs","repos_url":"https://api.github.com/users/JeniT/repos","events_url":"https://api.github.com/users/JeniT/events{/privacy}","received_events_url":"https://api.github.com/users/JeniT/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-04-08T20:29:02Z","updated_at":"2015-04-09T05:11:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/w3c/csvw/pulls/469","html_url":"https://github.com/w3c/csvw/pull/469","diff_url":"https://github.com/w3c/csvw/pull/469.diff","patch_url":"https://github.com/w3c/csvw/pull/469.patch"},"body":"Remove `property value` term, and use within the model document refering instead to the appropriate annotation value within the model document.\r\nFixes #463."},"comment":{"url":"https://api.github.com/repos/w3c/csvw/issues/comments/91117029","html_url":"https://github.com/w3c/csvw/pull/469#issuecomment-91117029","issue_url":"https://api.github.com/repos/w3c/csvw/issues/469","id":91117029,"user":{"login":"iherman","id":520723,"avatar_url":"https://avatars.githubusercontent.com/u/520723?v=3","gravatar_id":"","url":"https://api.github.com/users/iherman","html_url":"https://github.com/iherman","followers_url":"https://api.github.com/users/iherman/followers","following_url":"https://api.github.com/users/iherman/following{/other_user}","gists_url":"https://api.github.com/users/iherman/gists{/gist_id}","starred_url":"https://api.github.com/users/iherman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/iherman/subscriptions","organizations_url":"https://api.github.com/users/iherman/orgs","repos_url":"https://api.github.com/users/iherman/repos","events_url":"https://api.github.com/users/iherman/events{/privacy}","received_events_url":"https://api.github.com/users/iherman/received_events","type":"User","site_admin":false},"created_at":"2015-04-09T05:11:28Z","updated_at":"2015-04-09T05:11:28Z","body":"I have an unease but I am not sure I can put the finger on it very precisely.\r\n\r\nThis set of changes was mostly triggered by the approach that the model document and the conversions should stand on their own because other specifications may be created in the future that might create and annotated table through some other means than what is described in the metadata document. And if such specs are created, we do not want to redo, say, the metadata spec. I hope I understand this right.\r\n\r\nHowever, ideally, what this should mean is that the syntax document should actually *not* refer to the metadata document in any normative way. However, that would require a much more thorough re-write. Let me take two examples for my unease (referring to the version in the branch-to-be-merged).\r\n\r\nOn the one hand, the metadata document describes `aboutUrl` as a URI Template property with what we have before in section 4.7. On the other hand, the syntax document says (in 3.5):\r\n\r\n> **about URL** — a URL for the entity that this cell provides information about, or null. The value of this annotation is derived as described in (reference to section 4.7) \r\n  \r\nSurely than this should be changed, right? How the ``aboutUrl`` came into existence is irrelevant as far as the model document is concerned, other specifications may end up producing the URL by other means. The metadata document should not *define* an annotation value!\r\n\r\nAnother, similar, issue is around natural language properties (as we call them in the metadata document). The syntax document says\r\n\r\n> titles — any number of human-readable titles for the column, each of which has an associated language\r\n\r\nwhereas the metadata document describes them in much more operational detail:\r\n\r\n> The annotation value of a natural language property is an object whose properties are language codes and where the values of those properties are an array of strings (see Language Maps in [JSON-LD]).\r\n\r\nBut... is it o.k. to actually *define* the annotation value in the metadata document? Shouldn't the definition be part of the syntax document entirely? The language map structure is, in this sense, irrelevant for the syntax document, isn't it?\r\n\r\nIn general, I believe *all* references to the metadata document from the syntax document should be considered and, if possible, removed, more exactly replaced by the definition of the annotation value in that document.\r\n\r\nSorry if I am not entirely systematic in my rumbling..."}}