Skip to content

DOC: Make some formatting changes and add a "Using dat" section#371

Merged
okdistribute merged 3 commits intodat-ecosystem:masterfrom
waldoj:master
Aug 5, 2015
Merged

DOC: Make some formatting changes and add a "Using dat" section#371
okdistribute merged 3 commits intodat-ecosystem:masterfrom
waldoj:master

Conversation

@waldoj
Copy link
Copy Markdown
Contributor

@waldoj waldoj commented Aug 4, 2015

These two commits are isolated, with one making general improvements to the text, the other adding a new section entirely.

@okdistribute
Copy link
Copy Markdown
Collaborator

This is great! The exoplanets example could be changed to import directly from a url - dat handles that

@okdistribute
Copy link
Copy Markdown
Collaborator

@waldoj I guess what confuses me here is that you say to put the csv into the same directory, but really the dat should replace the csv, so you should import it to the dat from wherever it happens to be on your filesystem or on the web

@waldoj
Copy link
Copy Markdown
Contributor Author

waldoj commented Aug 4, 2015

I was writing this while also struggling (and failing) to install dat on my local machine, so I couldn't test it to see if that more efficient route would work. :) I've got that improved now. I sure don't think that this is a brilliant "Using dat" section, but I figure it's better than nothing, and at least starts to address the concern raised by @mlissner in #364, that there needs to be a quick way for somebody to be able to play with dat.

I'd really like to see this section instead fork an existing dataset, so somebody can try changing it, seeing a diff, etc. Baby steps. :)

@okdistribute
Copy link
Copy Markdown
Collaborator

lgtm!

okdistribute pushed a commit that referenced this pull request Aug 5, 2015
DOC: Make some formatting changes and add a "Using dat" section
@okdistribute okdistribute merged commit e4e3811 into dat-ecosystem:master Aug 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants