-
Notifications
You must be signed in to change notification settings - Fork 17
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
update scikit_learn version to make it work #34
Comments
thanx, it helped a lot! |
Hi,
Seems something has changed inbetween the sklearn versions. |
Hi, I managed to get mCaller working with sklearn ≥ 1 with the following patch. However if I recall correctly my python3 conventions, the '_' prefix denotes internal symbols, so it feels more like a work around than a real fix. Besides you might want to implement it in a more flexible way to support both former and later sklearn versions. Any ways, I hope this helps. Have a nice day, :) --- mcaller.orig/r94_model_NN_6_m6A.pkl
+++ mcaller/r94_model_NN_6_m6A.pkl
@@ -1,7 +1,7 @@
ccopy_reg
_reconstructor
p1
-(csklearn.neural_network.multilayer_perceptron
+(csklearn.neural_network._multilayer_perceptron
MLPClassifier
p2
c__builtin__
@@ -68,7 +68,7 @@
tbsS'_label_binarizer'
p20
g1
-(csklearn.preprocessing.label
+(csklearn.preprocessing._label
LabelBinarizer
p21
g3
--- mcaller.orig/r95_twobase_model_NN_6_m6A.pkl
+++ mcaller/r95_twobase_model_NN_6_m6A.pkl
@@ -4,7 +4,7 @@
ccopy_reg
_reconstructor
p3
-(csklearn.neural_network.multilayer_perceptron
+(csklearn.neural_network._multilayer_perceptron
MLPClassifier
p4
c__builtin__
@@ -71,7 +71,7 @@
tbsS'_label_binarizer'
p22
g3
-(csklearn.preprocessing.label
+(csklearn.preprocessing._label
LabelBinarizer
p23
g5 |
Hi, I needed to:
to make it run.
Since in the README there are no specifications for the versions, here are the versions that work on
python=3.8.8
:The text was updated successfully, but these errors were encountered: