-
Notifications
You must be signed in to change notification settings - Fork 2
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
data file documentation through fetchData() #2
Comments
Another option would be to take advantage of comment characters to make the files self documenting. That way there is only one file to deal with. In any case, I think fetching documentation should be a separate function (fetchDoc) instead of a flag. And do we know if anyone else besides Danny is using this feature? |
From @nicholasjhorton on October 20, 2012 13:3 By feature, do you mean using fetchData()? I am, but that's because it's necessitated by the instructions in the second edition of the textbook and the AcroScore problems. Overall, it works well for the built-in datasets. Nick On Oct 19, 2012, at 3:14 PM, Randall Pruim notifications@github.com wrote:
Nicholas Horton |
From @nicholasjhorton on March 18, 2013 19:46 We agreed to deprecate fetchData() in the next release (potentially over a 2 year period): this just gets added to the NEWS for now, but might eventually involve adding a warning when it is used. |
From @nicholasjhorton on March 19, 2013 10:43 Danny wrote: After falling asleep as soon as I got back to the room, I woke up at 10:30 with a simple idea to make fetchData() useful to instructors who want a simple way to post their own data.
I've prototyped this using the mosaic-web.org server, creating a NJH account which I've redirected to one of my non-fetchData() dropbox directories. (I don't have access to your server, but if you send me a link to a CSV file on your server, I'll update NJH to go to your server.) Here's the prototype: remoteFetch() in the attached file fetchRemote.R. Ignore the name; it would be folded into fetchData() Once you've sourced remoteFetch.R, you can try these commands with the files being served from a vanilla Dropbox directory on my account:
Regards, |
How close are we to |
Another stranded issue regarding |
From @nicholasjhorton on February 26, 2014 16:40 I’m increasingly frustrated with fetchData(), in that it makes it hard to reference datasets, since students will often interchange things like: ds = fetchData(“KidsFeet”) or kids = fetchData(“KidsFeet”) then wonder why their later commands don’t work (using the other name). Can we move to referencing dataframes from packages? Just my $0.02, Nick On Feb 26, 2014, at 11:09 AM, Randall Pruim notifications@github.com wrote:
Nicholas Horton |
I don't use I think use of If this is only useful as a way for Danny and a few others to distribute their data, then we should put it in a separate package, and document it as a tool for that more limited purpose. I'd like to get this resolved by the end of March. We've been dancing around this for too long. |
Seems we are still dancing around |
From @dtkaplan on October 19, 2012 18:47
I propose to add a
documentation=TRUE
argument tofetchData()
that will cause a corresponding documentation file to be display. Such files can give descriptions, etc. of the data. The author would upload a documentation file in an appropriate format.Copied from original issue: ProjectMOSAIC/mosaic#169
The text was updated successfully, but these errors were encountered: