ropensci / neotoma

Programmatic R interface to the Neotoma Paleoecological Database.
https://docs.ropensci.org/neotoma
Other
30 stars 16 forks source link

Add an extractor function for count data #146

Closed gavinsimpson closed 9 years ago

gavinsimpson commented 10 years ago

This issue is linked to #141 and relates to an extractor for count data held within neotoma objects.

This issue can be tracked in the add-counts-extractor branch of my fork.

SimonGoring commented 10 years ago

Awesome, I assume this means you're working on it. I appreciate it.

gavinsimpson commented 10 years ago

@SimonGoring Yes, though needed to solve that install error so I could run the code from the paper to check. I should have this finished over the weekend (meetings all day tomorrow) sufficient for interested people to comment on implementation etc.

SimonGoring commented 10 years ago

Cool. I think I've cleaned up most of my mess. Tomorrow I'm going to work on fixing up the calls in neotoma_paper and getting back on track with that, but I've got a date with a very adorable three year old that will take up most of my morning.

On Thu, Sep 25, 2014 at 11:16 PM, Gavin Simpson notifications@github.com wrote:

@SimonGoring https://github.com/SimonGoring Yes, though needed to solve that install error so I could run the code from the paper to check. I should have this finished over the weekend (meetings all day tomorrow) sufficient for interested people to comment on implementation etc.

— Reply to this email directly or view it on GitHub https://github.com/ropensci/neotoma/issues/146#issuecomment-56917630.

SimonGoring commented 9 years ago

Gavin, have you made any progress on this? I was thinking that we could try to add a method for Stratiplot from a download object, and make other more explicit links to analogue. Not sure if neotoma is the best place for that, or somewhere else, but the extractor function would help.

gavinsimpson commented 9 years ago

@SimonGoring yes, but needed to do a bit of fixing up in my add-counts-extractor branch to include all the changes made since we worked on this. Just checking it still works.

SimonGoring commented 9 years ago

This issue is resolved. Counts works nicely.