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

Checking all installs #705

Closed
bioforkgithub opened this issue Jul 4, 2023 · 11 comments
Closed

Checking all installs #705

bioforkgithub opened this issue Jul 4, 2023 · 11 comments

Comments

@bioforkgithub
Copy link

python3 -c 'import sklearn' NOT OK
Missing python library "sklearn"

I have installed the scikit-learn package but still it shows this error on running the test_install.pl

@bioforkgithub
Copy link
Author

I tried both the packages mamba as well as the git install

@fpusan
Copy link
Collaborator

fpusan commented Jul 4, 2023

What's the output when running python3 -c 'import sklearn' inside the activated conda environment?

@bioforkgithub
Copy link
Author

  1. micromamba list gives following:

scikit-learn 0.24.2 py36hc89565f_1 conda-forge
scipy 1.5.3 py36h81d768a_1 conda-forge
sed 4.8 he412f7d_0 conda-forge
setuptools 58.0.4 py36h5fab9bb_2 conda-forge
six 1.16.0 pyhd3eb1b0_1
sqlite 3.40.1 h5082296_0
squeezemeta 1.6.2post1 py36pl5262r36h7b7c402_0 fpusan

  1. python3 -c 'import sklearn' gives following:

(SqueezeMeta) [a40540@hod localscratch]$ python3 -c 'import sklearn'
Traceback (most recent call last):
File "", line 1, in
File "/gpfs/gpfs0/home/home2/a40540/micromamba/envs/SqueezeMeta/lib/python3.6/site-packages/sklearn/init.py", line 82, in
from .base import clone
File "/gpfs/gpfs0/home/home2/a40540/micromamba/envs/SqueezeMeta/lib/python3.6/site-packages/sklearn/base.py", line 17, in
from .utils import _IS_32BIT
File "/gpfs/gpfs0/home/home2/a40540/micromamba/envs/SqueezeMeta/lib/python3.6/site-packages/sklearn/utils/init.py", line 23, in
from .class_weight import compute_class_weight, compute_sample_weight
File "/gpfs/gpfs0/home/home2/a40540/micromamba/envs/SqueezeMeta/lib/python3.6/site-packages/sklearn/utils/class_weight.py", line 7, in
from .validation import _deprecate_positional_args
File "/gpfs/gpfs0/home/home2/a40540/micromamba/envs/SqueezeMeta/lib/python3.6/site-packages/sklearn/utils/validation.py", line 22, in
import joblib
File "/gpfs/gpfs0/home/home2/a40540/micromamba/envs/SqueezeMeta/lib/python3.6/site-packages/joblib/init.py", line 129, in
from .parallel import Parallel
File "/gpfs/gpfs0/home/home2/a40540/micromamba/envs/SqueezeMeta/lib/python3.6/site-packages/joblib/parallel.py", line 23, in
from contextlib import nullcontext
ImportError: cannot import name 'nullcontext'

Even with the error I tried running the pipeline with test data. It ran without error but does it affect the credibility of result???????

@fpusan
Copy link
Collaborator

fpusan commented Jul 9, 2023

Not sure which package uses scikit, your results are probable fine if there was no error message.
Anyways I am working on a Foix for this
You can try mamba create -n SqueezeMeta -c conda-forge -c bioconda -c fpusan -c anaconda squeezemeta-dev=1.6.2post2 --no-channel-priority to install the de versiin with the Fox. It should pass the tests in test_install.py. I will do more testing myself and hopefully release the patch next week

@bioforkgithub
Copy link
Author

Ok then I will do that next week and try again on the test data. Rightnow I am running it on my data. Waiting for any errors.
Will let you know for any error then.

@fpusan
Copy link
Collaborator

fpusan commented Jul 11, 2023 via email

@RIJP-SU
Copy link

RIJP-SU commented Jul 11, 2023

Sorry I should have cheked your dev version befor posting. It worked fine. Thanks.

*I accidentally deleted follwoing message. Applogize for making mess.

Hi, I've got same message from test_install.pl. I think version of scikit-learn is not compatible for python 3.6. When installing squeezemeta with mamba, scikit-learn version 0.24.2 is installed. However this version is not compatible for python 3.6. I tried downgrading scikit-learn to 0.20.2 which is compatible for python 3.6, but failed due to dependency problems. Any solution for this?

@fpusan
Copy link
Collaborator

fpusan commented Jul 11, 2023

No, it's fine. Dev versions are not normally meant for downloading, sometimes we have some non-working experimental stuff in there

@fpusan
Copy link
Collaborator

fpusan commented Jul 11, 2023

Should be fixed in ba9a595 and the new release v1.6.2post2

@fpusan fpusan closed this as completed Jul 11, 2023
@bioforkgithub
Copy link
Author

bioforkgithub commented Jul 19, 2023

The execution is complete but i did not get any error messages for scikit. Seems everything is fine. This is from the previous install,not the updated one you mentioned. Will try the next one as well. If any errors, i will keep you posted

@fpusan
Copy link
Collaborator

fpusan commented Jul 19, 2023

Even if you got no error messages, CONCOCT binning (if requested) may have failed. In that case the pipeline would have progressed using the results from the other binning methods

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

No branches or pull requests

3 participants