-
Notifications
You must be signed in to change notification settings - Fork 309
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
LinearRegression unknown keyword normalize #506
Comments
Thank you!! This is a great find. I'll get on it soon. Since |
Hi, I just wanted to mention that the version that you have on Conda-forge is not up to date, and results on the same issues. Just that. |
Hi, Is there any (quick) fix to this available? |
@ddeboerfluid The quick fix is to install an updated version, either from pypi or here. Actually, it looks like this bug was fixed at least two years ago, which is where the blame of the most recent normalization code points (b9fc48c). We'll be releasing a new version shortly, but you can always get the current version with
@ggmirandac, it was only put on conda-forge a few years ago manually by a previous maintainer. I don't personally use conda, but I would be happy for a PR into .github/workflows/release.yml to automate that release. |
from sklearn.linear_model import LinearRegression lr = LinearRegression() can also works |
I just installed the PySINDy library, and it throws an
TypeError: LinearRegression.__init__() got an unexpected keyword argument 'normalize'
out of the box.The version that conda installed for
scikit-learn
without any specification is1.4.2
.According to the Changelog from
scikit-learn
, thenormalize
keyword has been deprecated since version1.0
and removed since1.2
. However there is a proposed fix in the above linked changelog.Reproducing code example:
Error message:
PySINDy/Python version information:
1.3.0 3.10.14 (main, May 6 2024, 19:42:50) [GCC 11.2.0]
The text was updated successfully, but these errors were encountered: