search

pa-m / sklearn

bits of sklearn ported to Go #golang
MIT License
345 stars 38 forks source link

pa-m:sklearn - Changed maxFev param type from int to int64 for 32-bit builds. #8

Closed edstardo closed 4 years ago

edstardo commented 5 years ago

I'm using pa-m:sklearn/preprocessing StandardScaler and I get this error when trying to build for x86:

To fix this, I have modified 3 files in pa-m:optimize and changed fnMaxFev param from type int to type int64 which is already in a separate PR

I also changed sklearn:preprocessing/data.go

After these changes I can successfully build for x86 without problems. All tests are also passing.

pa-m commented 4 years ago

Hi, powell's minimization has been moved out of sklearn because it's not part of scikit-learn. It's available in https://github.com/pa-m/optimize.