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

_nanrankdata #8

Closed
nils-braun opened this issue May 18, 2016 · 2 comments
Closed

_nanrankdata #8

nils-braun opened this issue May 18, 2016 · 2 comments

Comments

@nils-braun
Copy link

Thank you for your nice implementation of boruta!
However, I am wondering why you reimplemented the nanrankdata method of bottleneck. The reason I am wondering is that your new implementation can not handle cases, where the input data is in the form

X = [..., [nan, nan, nan], ...]

whereas the bottleneck.nanrankdata can (and this case occurs in my examples). I am sure there was a good reason to not use this method from bottleneck....

Thank you for your help!

@danielhomola
Copy link
Collaborator

Thanks for letting me know! I love bottleneck and use it all the time, but I also want to add Boruta to scikit-learn and they have a very strict dependency policy so I had to re-implement nansort with numpy/scipy. Do you have an idea how we could fix the issue you found?

Cheers,
d

@nils-braun
Copy link
Author

Ah ok, now I understand why you switched from bottleneck to your implementation.
No, I do not have a quick fix in mind. I will think about it and answer, when I have found something.

Cheers
Nils

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