Skip to content
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

Pre-processing to remove non-numeric columns #2

Open
mpadge opened this issue Sep 2, 2021 · 1 comment
Open

Pre-processing to remove non-numeric columns #2

mpadge opened this issue Sep 2, 2021 · 1 comment

Comments

@mpadge
Copy link

mpadge commented Sep 2, 2021

I've started playing around with this package for my own work, for which it's looking really promising. Also opens up a very interesting "can-o-worms" regarding justification of which ordination method one chooses 😟 ... but that discussion can be held some other time and place. In the meantime, as a first observation:

It would be useful to have a couple of pre-processing steps to make the package more "user friendly." One would be to do a simple check for any non-numeric columns and reduce the input object, xx, down to numeric columns only. This is done in most stats routines including cmdscale, prcomp, and the like, which just issue warnings yet still process the data. dobin currently errors with very uninformative message, Error in max.z - min.z : non-numeric argument.

@sevvandi
Copy link
Owner

sevvandi commented May 18, 2022

Thanks Mark. I've done the pre-processing to remove non-numeric values. I also added a non-normalising option.

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

No branches or pull requests

2 participants