-
-
Notifications
You must be signed in to change notification settings - Fork 262
Consider using hypothesis for testing. #17
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
Comments
Cool, I've wanted to try it out on a real project for years. |
Great. I'll use it in whatever I try to do next (after the Imputer, which I don't think makes much sense). Maybe we can start making some estimators? |
By all means. Which ones are you interested in working on? |
I think it would be strategic to implement ALS. Spark has gained some spectacular popularity in recommendation systems due to it. There's an implementation of a somewhat similar method in sklearn, but I think it would be wise to go rogue on this one. I've implemented it before by hand using the multiprocessing lib in python in the past. Should be quite easy to migrate it. What do you think? |
ALS seems like an ideal candidate. |
Hi I've also would like to play with hypothesis. How can I help? @daniel-severo can you point to where you had started to play with it? |
I'm not familiar with using hypothesis, so I may be wrong, but I think it'd
be more straightforward to apply it
to pandas itself, rather than dask-ml. I opened
pandas-dev/pandas#17978 to
discuss using it in pandas.
…On Tue, Oct 24, 2017 at 12:00 PM, Joan Massich ***@***.***> wrote:
Hi I've also would like to play with hypothesis. How can I help?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#17 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABQHIid9isk5QL9WfjqMBNSxuFbTP7GNks5svhelgaJpZM4Pv6yH>
.
|
Maybe we should consider using hypothesis for testing. I use it here at work to test ETL and data pipelines and it works like a charm.
The text was updated successfully, but these errors were encountered: