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

[MOD] Bug Fix for sklearn 1.0~ #19

Merged
merged 1 commit into from Oct 3, 2021
Merged

Conversation

CryptoSalamander
Copy link
Contributor

@CryptoSalamander CryptoSalamander commented Oct 3, 2021

scikit-learn has been updated to 1.0.0. As a result, there are some functions that don't work properly. it makes errors like the below:

TypeError: __init__() takes from 1 to 2 positional arguments but 5 were given.

To fix this problem, I added * in init parameters refers to PEP 3102(https://www.python.org/dev/peps/pep-3102/).

@CryptoSalamander
Copy link
Contributor Author

@trent-b I fixed some errors due to scikit-learn's new release version. could you check it?

@trent-b
Copy link
Owner

trent-b commented Oct 3, 2021

Thank you, @CryptoSalamander!

@trent-b trent-b merged commit d5f6192 into trent-b:master Oct 3, 2021
@trent-b
Copy link
Owner

trent-b commented Oct 3, 2021

@CryptoSalamander, the extra "*" parameters was in the wrong place, but I fixed it, and it's all good now.

@CryptoSalamander
Copy link
Contributor Author

@CryptoSalamander, the extra "*" parameters was in the wrong place, but I fixed it, and it's all good now.

Thank you sir!

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

Successfully merging this pull request may close these issues.

None yet

2 participants