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

Convert to datapackage #144

Closed
patcon opened this issue Feb 5, 2017 · 6 comments
Closed

Convert to datapackage #144

patcon opened this issue Feb 5, 2017 · 6 comments

Comments

@patcon
Copy link

patcon commented Feb 5, 2017

Would this be a sensible thing to do? Seems like this fits the normal format of scripts + data. Would a PR for this be accepted?

Seems like it might be worth the exercise, as a repo with some many data files would probably bring up pain points in the datapackage toolset worth addressing.

@jpmckinney
Copy link
Member

Most users just use the single compiled CSV for their country, which is easy to maintain by just downloading the latest version. Perhaps a third-party data package could be maintained?

@patcon
Copy link
Author

patcon commented Feb 6, 2017

You mean one per country? Or still one with all countries, but just the CSVs and metadata?

@jpmckinney
Copy link
Member

@patcon
Copy link
Author

patcon commented Feb 6, 2017

Ah gotcha. I'll admit that one thing I really liked about the datapackage philosophy was that it seemed to encourage the co-location of the data and the scripts from which the data is massaged/derived.

Any preference on that yourself, or are you just as happy to have the data live on its own in a datapackage repo?

@jpmckinney
Copy link
Member

Isn't this repo already in a situation where the scripts and data are collocated? (They are in the same repo.)

@patcon
Copy link
Author

patcon commented Feb 6, 2017

Def. The original ask was whether it would be worthwhile to add a datapackage.json describing the data :)

But I suppose until this data is in a CKAN portal somewhere, having that metadata isn't much value-add for the effort. So I'll just close until such a time. Sorry for the noise on this one

@patcon patcon closed this as completed Feb 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants