{"action":"created","issue":{"url":"https://api.github.com/repos/w3c/csvw/issues/679","labels_url":"https://api.github.com/repos/w3c/csvw/issues/679/labels{/name}","comments_url":"https://api.github.com/repos/w3c/csvw/issues/679/comments","events_url":"https://api.github.com/repos/w3c/csvw/issues/679/events","html_url":"https://github.com/w3c/csvw/issues/679","id":94436754,"number":679,"title":"Editorial comments on the metadata document","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},"labels":[{"url":"https://api.github.com/repos/w3c/csvw/labels/CSV%20to%20JSON%20mapping","name":"CSV to JSON mapping","color":"84b6eb"},{"url":"https://api.github.com/repos/w3c/csvw/labels/CSV%20to%20RDF%20mapping","name":"CSV to RDF mapping","color":"009800"},{"url":"https://api.github.com/repos/w3c/csvw/labels/Editor%20action","name":"Editor action","color":"5319e7"},{"url":"https://api.github.com/repos/w3c/csvw/labels/For%20Proposed%20Rec","name":"For Proposed Rec","color":"006b75"},{"url":"https://api.github.com/repos/w3c/csvw/labels/Metadata%20vocabulary%20document","name":"Metadata vocabulary document","color":"5319e7"},{"url":"https://api.github.com/repos/w3c/csvw/labels/NonWG","name":"NonWG","color":"fad8c7"}],"state":"open","locked":false,"assignee":{"login":"6a6d74","id":2519144,"avatar_url":"https://avatars.githubusercontent.com/u/2519144?v=3","gravatar_id":"","url":"https://api.github.com/users/6a6d74","html_url":"https://github.com/6a6d74","followers_url":"https://api.github.com/users/6a6d74/followers","following_url":"https://api.github.com/users/6a6d74/following{/other_user}","gists_url":"https://api.github.com/users/6a6d74/gists{/gist_id}","starred_url":"https://api.github.com/users/6a6d74/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/6a6d74/subscriptions","organizations_url":"https://api.github.com/users/6a6d74/orgs","repos_url":"https://api.github.com/users/6a6d74/repos","events_url":"https://api.github.com/users/6a6d74/events{/privacy}","received_events_url":"https://api.github.com/users/6a6d74/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/w3c/csvw/milestones/1","html_url":"https://github.com/w3c/csvw/milestones/Proposed%20Rec","labels_url":"https://api.github.com/repos/w3c/csvw/milestones/1/labels","id":1203242,"number":1,"title":"Proposed Rec","description":null,"creator":{"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},"open_issues":8,"closed_issues":0,"state":"open","created_at":"2015-07-11T04:33:28Z","updated_at":"2015-07-30T10:39:34Z","due_on":null,"closed_at":null},"comments":4,"created_at":"2015-07-11T04:54:41Z","updated_at":"2015-08-02T16:32:19Z","closed_at":null,"body":"(This is a transfer of the [comments sent by 'timeless'](http://www.w3.org/mid/CACsW8eH8iGOmGsbOc8L96M-Y99LA2uUXbwgCmmtchSa-cZACpA@mail.gmail.com) to public-csv-wg-comments@w3.org')\r\n\r\nW3C Editor's Draft 10 July 2015 [1]\r\n>\r\n>We are on Github\r\n\r\nGitHub\r\n\r\n>Data Aggregators can use the indicated metadata, such as descriptions, titles, modification dates, and licences [sic], to enable more intelligent retrieval of relevant data on the web.\r\n\r\n>Implementations may fulfil [sic] one or more of these functions.\r\n\r\n>which are not defined in this specification and MUST generate a warning when they are encoutered [sic].\r\n\r\n>then that string is assumed to be in the default language (or have an undefined language, und, if there is no such property).\r\n\r\npattern:\r\nCould you write \"und\" instead of und (unquoted)? For those of us\r\nreading the document with a Screen Reader or similar tool (actually a\r\nplain text serializer in my case), the monospace formatting is lost.\r\n\r\n>Foreign key definitions provide for strong linking between tables that guarantees (through validation) the existance [sic] of a referenced row.\r\n\r\n>For example, if an implementation recognises and displays the value of the dc:description property, it should also recognise and display the value of the http://purl.org/dc/terms/description property in the same way.\r\n\r\nen-US:\r\nplease see point 2 of [2]\r\n\r\n>Note that this format is not the same as the existing text/csv and text/tab-delimited-values mediatypes [sic],\r\n\r\nGoogle says it's two words.\r\n\r\n>The CSV on the Web Working Group was chartered to produce a\r\n\r\nThis is the first instance of the group, but it isn't linked here\r\n(it's linked in the next paragraph). That feels like putting the cart\r\nbefore the horse.\r\n\r\nAlso, if you're going to mention the charter, you could do everyone\r\nthe favor of linking to it.\r\n\r\n>Recommendation \"Access methods for CSV Metadata\" as well as\r\n>Recommendations for \"Metadata vocabulary for CSV data\" and \"Mapping mechanism to transforming CSV into various Formats (e.g., RDF, JSON, or XML)\"\r\n\r\nI object to capitalizing `Formats`.\r\n\r\n>This document aims to primarily satisfy the second of those Recommendations.\r\n\r\nI don't think that word should be capitalized in this context.\r\n\r\nDepending on how you parse the previous lines, this either references\r\n2/2 or 2/3.\r\n\r\nI'm not quite sure the value of writing this as you did. I'd suggest\r\n\"This document addresses the charter scope for {}. Additional charter\r\nscope items include {} and {} which would be covered by other\r\ndocuments.\"\r\n\r\n>As an example, say that the following CSV file were available at http://example.org/tree-ops.csv\r\n\r\nmissing `:`\r\n\r\n>For example, say that the following metadata file were available at http://example.org/tree-ops.csv-metadata.json:\r\n\r\n>[reference]\r\n>A document reference (normative or informative) is enclosed in square brackets and links to the references section.\r\n\r\nHere, the brackets are included in the link, they aren't in the document.\r\n\r\n>compliant applications MUST use that default value and MUST generate a warning.\r\n\r\n=> MUST generate a warning and use that default value.\r\n\r\nwhich parallels this:\r\n\r\n>If no default value is provided for that property, compliant applications MUST generate a warning and behave as if the property had not been specified.\r\n\r\n>including properties (aside from common properties) which are not defined in this specification.\r\nhaving invalid values for a given property.\r\n\r\nthese items are not sentences and aren't preceded by a `:`\r\n\r\n>on a [sic] annotated tabular data model.\r\n\r\n\"an\"\r\n\r\n>A description object is a JSON object that describes a component of the annotated tabular data model (a group of tables, a table or a column) and has one or more properties are mapped into properties on that component.\r\n\r\n`are` => that are | {empty}\r\n\r\n>For example, in the column description\r\n\r\nadd `,` or `:` ?\r\n\r\nOffhand, figure 1 [3] was generated at a resolution which is too small\r\nto read at 100%; text runs together... Also, the gray background is\r\nunhelpful.\r\n\r\nfigure 1 [4] isn't accessible, it's just a bunch of lines / polylines.\r\nSomeone helpfully stripped all the words from the source code.\r\n\r\n>There are different types of properties on description objects:\r\n>* section 5.1.1 Array Properties.\r\n>* section 5.1.2 Link Properties.\r\n\r\nthese aren't sentences...\r\n\r\n>The variables that are set are:\r\n\r\n... are ... are <- yuck\r\n\r\nmost of these don't end in a period:\r\n>_sourceRow is set to the source number of the row that is currently being processed; this usually varies from _row by skip rows and header rows\r\n\r\nexcept this one:\r\n>_name is set to the URI decoded column name annotation, as defined in [tabular-data-model], for the column that is currently being processed. (Percent-decoding is necessary as name may have been encoded if taken from titles; this prevents double percent-encoding.)\r\n\r\n>The propertyUrl property might be defined as \"{#_name}\", meaning that it resolves as a fragment identifier relative to the URL of the source of the table. For example, accessing it from a column with the column name GID would look like:\r\n\r\nI couldn't follow this example. Partially, I think you hid the input\r\nas prose and only showed output in example 11, whereas example 10 was\r\nthe input and the prose held the output.\r\n\r\n>the processor will issue a warning and ignore the value.\r\n\r\npattern:\r\n`will` isn't an rfc word\r\n\r\nNo Oxford Comma:\r\n>An atomic property that MUST have a single string value that is one of \"rtl\", \"ltr\" or \"auto\".\r\n\r\nOxford Comma:\r\n>Indicates whether the tables in the group should be displayed with the first column on the right, on the left, or based on the first character in the table that has a specific direction.\r\n\r\nplease pick a style (I'd recommend Oxford).\r\n\r\n>It is also possible to provide weak linking between tables that are not tested by validations but which may be useful when converting tabular data into other formats, using aboutUrl and valueUrl.\r\n\r\nI think you want to drop the `,`;\r\n\r\nthe sentence is too long.\r\n\r\n>This annotation MUST be percent-encoded as necessary to conform to the syntactic requirements defined in [RFC3986]\r\n\r\nmissing `.`\r\n\r\n>A boolean atomic property taking a single value\r\n\r\nisn't `taking a single value` redundant for `boolean`?\r\n\r\n>a [sic] inherited property defined in its containing schema description\r\n\r\nan\r\n\r\n>takes precedence of one defined in its containing table description\r\n\r\npattern:\r\nof => over\r\n\r\n>This property is irrelevant if the separator is null or undefined, but this is not an error.\r\n\r\n`but this is not an error` doesn't make sense / read well\r\n\r\n>An [sic] URI template property that MAY be used to create a URI for a property if\r\n\r\npattern:\r\nA -- note that you use `a URI` elsewhere, including later in this fragment.\r\n\r\n>quoteChar\r\n>The default is \".\r\n\r\nI'd suggest writing this as: \"\\\"\" or '\"'\r\n\r\n>An [sic] boolean atomic property\r\n>An [sic] numeric atomic property\r\n\r\npattern:\r\nA --\r\n\r\n>A boolean atomic property that, if true, sets the trim flag to \"start\".\r\nIf false, to false.\r\n\r\nThis second \"sentence\" needs more words.\r\nIt probably should be `to \"false\"` also...\r\n\r\n>No other values are valid.\r\n\r\nIs that an \"Error\", or just \"whatever\"?\r\n\r\n>Applications MUST raise an error if length, maxLength, or minLength are specified and the base datatype is not string or one of its subtypes, or a binary type.\r\n\r\nthe order of `or`s doesn't work.\r\n\r\ntry ... neither string/string subtype nor a binary type.\r\n\r\n>Values MAY be a string, native JSON type (such as number, true, or false.), value object, node object or an array of zero or more of any of these.\r\n\r\nstray `.` inside the parenthetical\r\n\r\n\r\n\r\n[1] http://w3c.github.io/csvw/metadata/\r\n[2] https://lists.w3.org/Archives/Public/public-csv-wg-comments/2015Jun/0002.html\r\n[3] http://w3c.github.io/csvw/metadata/properties.png\r\n[4] http://w3c.github.io/csvw/metadata/properties.svg"},"comment":{"url":"https://api.github.com/repos/w3c/csvw/issues/comments/127042338","html_url":"https://github.com/w3c/csvw/issues/679#issuecomment-127042338","issue_url":"https://api.github.com/repos/w3c/csvw/issues/679","id":127042338,"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-08-02T16:32:19Z","updated_at":"2015-08-02T16:32:19Z","body":"To follow up on my [previous note](https://github.com/w3c/csvw/issues/679#issuecomment-120573022), I have added a [longdesc file to describe the property diagram](http://w3c.github.io/csvw/metadata/properties.html), taking care of one of the (sub-)issues."}}