Behavior of taxmap$func() vs func(taxmap) about taxa

[sckott]

I agree that any one person will likely stick to one way or the other.

I think we should stick with what you've done for now - we should get others to take a look at the entire package soonish and if there's a strong feeling about changes, we can talk about that then

[zachary-foster]

Sounds good

[zachary-foster]

Do we still want to have others weigh in on this? I can ask a few coworkers.

[sckott]

yeah, that'd be great

[zachary-foster]

Hey @knausb and @Tabima, would you mind weighing in on this if you have time? Thanks!

[zachary-foster]

Oh yea, and we would of course like to hear from @grunwald as well.

[knausb]

I think its great that you have engineered both behaviours! I think that will make the package approachable to those of us who have a more more 'base R' perspective as well as those who want the side effects version.

[zachary-foster]

Thanks for the feedback @knausb!

[grunwald]

I agree with all comments and support the dual approach.

[zachary-foster]

Think we can close this or do we want more input?

[sckott]

i think we can close

[zachary-foster]

Cool