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
Adbscan fix #120
Adbscan fix #120
Conversation
Thank you for the review @weikang9009 @darribas! This should now be ready to merge. |
Not sure why the additional commits were introduced. Possibly through an erroneous rebase? Still, should not affect the core of the submission on adbscan.py? |
This should pass once #124 is merged. |
This makes
adbscan
(formally) a scikit-learn estimator by making its inheritance from_ClusterMixin
andBaseEstimator
explicit.I don't think this affects anything, but it'd be good to check more than the existing test suite...