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

Example 8 has tableSchema referring to CSV file #854

Open
mgoodhand opened this issue Aug 18, 2017 · 6 comments · May be fixed by #856
Open

Example 8 has tableSchema referring to CSV file #854

mgoodhand opened this issue Aug 18, 2017 · 6 comments · May be fixed by #856

Comments

@mgoodhand
Copy link

Example 8 in section 6 of "Model for Tabular Data and Metadata on the Web" shows a tableSchema pointing to a CSV file:

{
  "tableSchema": "government-spending.csv",
  "tables": [{
    "url": "http://example.org/east-sussex-2015-03.csv",
  }, {
    "url": "http://example.org/east-sussex-2015-02.csv"
  }, ...
  ]
}

https://www.w3.org/TR/2015/REC-tabular-data-model-20151217/#h-processing-tables

Is this intentional?

From my reading of "Metadata Vocabulary for Tabular Data" section 5.3.2, I'd expect either an embedded JSON object or a string referring to a JSON file:

An object property that provides a single schema description as described in section 5.5 Schemas, used as the default for all the tables in the group. This may be provided as an embedded object within the JSON metadata or as a URL reference to a separate JSON object that is a schema description.

https://www.w3.org/TR/2015/REC-tabular-metadata-20151217/#optional-properties

gkellogg added a commit that referenced this issue Aug 19, 2017
@gkellogg gkellogg linked a pull request Aug 19, 2017 that will close this issue
@gkellogg
Copy link
Member

I think this should be government-spending.json instead of government-spending.csv, but surrounding text does indicate getting metadata from CSV files as well. I believe this was @JeniT's example, and she may remember better. It certainly wouldn't be wrong to use the JSON reference, and would be less confusing.

@iherman
Copy link
Member

iherman commented Aug 20, 2017

Summary: a separate #856 PR has been issued, based on #854 (comment).

@mgoodhand
Copy link
Author

Thanks for the prompt review and fix.

@eddyracer
Copy link

Hello Hello

@eddyracer
Copy link

Looking for Government t

@eddyracer
Copy link

eddyracer commented May 1, 2021

Look for economic impact payment stimulus plus up payment deposits directly into my PayPal Account which is ready to accept deposits. As well as my Navy Federal Credit Union accounts.

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.

6 participants