-
Notifications
You must be signed in to change notification settings - Fork 18
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
idiv2: to do list #21
Comments
Folks keep adding things to the list of ToDo as you encounter them in the code |
@dmcglinn I feel responsible for the boxplots and for a vignette for mobr. However, with the latter I would like to wait until the normal function documentation is there. So let me know when you documented the functions. I will only be able to devote time, after November 9, which means after our iDiv conference |
Hey @dmcglinn - sorry about the delay! My family are in town, so I'm a bit lagging behind. If I remember correctly, there are a few things that we have agreed not to include, at least not in this first release:
Anything else? |
Hey @dmcglinn, I will take care of the documentation for the mob_stats and the boxplots function. However, so far I only worked on the master branch. Should I add the new stuff to the 4cur branch? Does that mean I also have to fork this branch or can I chose when I push and pull to which branch this happens? Thanks! |
Hey Felix
I'm going to merge 4cur with master today after Xiao has a chance to approve my PR. So you'll be able to just work with master shorty.
Dan
… On Nov 29, 2016, at 3:51 AM, Felix May ***@***.***> wrote:
Hey @dmcglinn, I will take care of the documentation for the mob_stats and the boxplots function. However, so far I only worked on the master branch. Should I add the new stuff to the 4cur branch? Does that mean I also have to fork this branch or can I chose when I push and pull to which branch this happens? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Hi Dan,
Yeah cool. Then I am in again – and I have no more excuses of not contributing ;o)
Felix
Von: Dan McGlinn [mailto:[email protected]]
Gesendet: Dienstag, 29. November 2016 12:20
An: MoBiodiv/mobr <[email protected]>
Cc: May, Felix <[email protected]>; Mention <[email protected]>
Betreff: Re: [MoBiodiv/mobr] idiv2: to do list (#21)
Hey Felix
I'm going to merge 4cur with master today after Xiao has a chance to approve my PR. So you'll be able to just work with master shorty.
Dan
On Nov 29, 2016, at 3:51 AM, Felix May ***@***.******@***.***>> wrote:
Hey @dmcglinn, I will take care of the documentation for the mob_stats and the boxplots function. However, so far I only worked on the master branch. Should I add the new stuff to the 4cur branch? Does that mean I also have to fork this branch or can I chose when I push and pull to which branch this happens? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#21 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AO8i0XO4ErVX85_7Uifr1EZQ2CHmVrDmks5rDApHgaJpZM4JFAVD>.
|
I just added additional items from the Felix to do list to a more general todo list for @rueuntal and myself |
rarefaction
so it is clear what the different methods accomplishsubset.mob_in
function that aids in pulling out a reduced number of rows of themob_in
object - this can be really useful when wanting to only work with certain treatment combinationsplot_rarefy
and move that code intoplot.mob_out
which currently calls that function.log
argument inplot.mob_out
as inmob_out
plot.mob_out
sometimes log transforms the x axis depending onmob_out$log_scale
and sometimes does not. This needs to be made consistent across the figures. (high priority)rarefaction
so it will accept a vector for sample based rarefactionmobr
object output fromget_delta_stats
Extrapolating the rarefaction curve because the number of rescaled individuals is smaller than the inds argument "effort" larger than total number of samples
rarefaction
that sample-based rarefaction is not actually used anywhere in the mob methods.The text was updated successfully, but these errors were encountered: