nok / sklearn-porter

Transpile trained scikit-learn estimators to C, Java, JavaScript and others.
BSD 3-Clause "New" or "Revised" License
1.28k stars 170 forks source link

Fix sklearn 0.20 module imports #81

Closed ghost closed 2 years ago

ghost commented 3 years ago

The sklearn imports are not longer working with sklearn version higher then 0.20

jona-sassenhagen commented 3 years ago

@nok any chance this will be approved and merged soon?

Piicksarn commented 2 years ago

Hi @nok :) would you mind to approve this request? It's helpful for the latest version of sklearn. thanks and regards

danielagottesman commented 2 years ago

any chance this can be merged?

nok commented 2 years ago

Hello, thanks for the hint.

@danielagottesman Yes, I will update and change it. In the next release you will be able to use always the latest version of scikit-learn. But the disadvantage will be that I can't guarantee changes related to the algorithms so I will print a warning. Finally the user will and can decide which version will be used for porting.

Kind regards, Darius