-
Notifications
You must be signed in to change notification settings - Fork 14
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
nosetests molmod fails #23
Comments
@wk858834 I'm sorry I missed this. I believe this is due to an API change in NumPy compared to the NumPy version that the conda package was built with. Did you install NumPy via Pip? That is a common cause for this problem and it often happen unintantionally, e.g. when using pip to install some package that requires a newer NumPy version than the one that comes with conda. |
I also encountered this error today when attempting to add Tamkin to an existing environment, but it was resolved by creating a new environment and installing Tamkin there.
In my case, numpy was definitely not installed using pip and was installed using conda. It may be that the version of numpy that was already present in the environment is incompatible with the one used to build tamkin, but shouldn't it then be the case that the version of numpy should be specified by the tamkin package? Or did a numpy update break backwards compatibility? |
TAMkin itself is not the problem because it is a pure Python code. This issue is specific for cython extensions, which are used in MolMod. How did you install MolMod? |
I installed molmod via conda.
Am Di., 3. März 2020 um 13:26 Uhr schrieb Toon Verstraelen <
notifications@github.com>:
… TAMkin itself is not the problem because it is a pure Python code. This
issue is specific for cython extensions, which are used in MolMod. How did
you install MolMod?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#23?email_source=notifications&email_token=ACQS6HM3K2MUPO6ZCVEBXKDRFTZPXA5CNFSM4JWOCOT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENTJINY#issuecomment-593925175>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACQS6HK6GN57JTB65ZVSAY3RFTZPXANCNFSM4JWOCOTQ>
.
|
I tried to reproduce this, but for me it all works. These are the steps taken: # skip the following if you already have conda
wget https://repo.continuum.io/miniconda/Miniconda3-latest-Linux-x86_64.sh -O miniconda3.sh;
bash miniconda3.sh -b -p $HOME/miniconda3
source ~/miniconda3/bin/activate
# Actual test
conda create -n tamkin python=3.7 nose
conda activate tamkin
conda install -c molmod tamkin
nosetests -v tamkin
# clean up
conda deactivate
conda remove --name tamkin --all One potential issue could be a mixup of a nose version installed outside conda (e.g. in |
In my functioning, dedicated TAMkin environment (python 2.7), I have the following versions based on conda list:
In my other environment, where I attempted to add TAMkin, nosetests still fail and the envionment is python 3.7:
|
I've modified my test procedure to better mimic your environment, switching of Miniconda2 for the base environment: # skip the following if you already have conda
wget https://repo.continuum.io/miniconda/Miniconda2-latest-Linux-x86_64.sh -O miniconda2.sh;
bash miniconda2.sh -b -p $HOME/miniconda2
source ~/miniconda2/bin/activate
# Actual test
conda create -y -n tamkin python=3.7 nose
conda activate tamkin
conda install -y -c molmod tamkin
nosetests -v tamkin
# clean up
conda deactivate
conda remove --name tamkin --all This still works on my side, so I'll suggest some things that might explain the different observations:
conda activate ChemEnv
conda update --all (Such upgrades could be potentially dangerous, so to play it safe, you could also make a new conda environment to test this.) |
|
Ok, there could be other packages preventing the upgrade of NumPy. This can be checked as follows: conda install numpy=1.18.1 If other packages require a lower version of NumPy, this will give an error message. |
Yes - I set up a tamkin environment with python 3 as above and that worked fine, but I ran into errors when then trying to add cantera to the environment. Output is attached in case it's of interest |
I see. I should update the NumPy dependency of the molmod package to be require at least version 1.16.0, which is the lowest version without this issue. This will effectively make it impossible to install it together with Cantera, alas. See e.g. GeoStat-Framework/GSTools#84 (comment) |
Fixed in #30 |
Dear Toon,
when installing conda install -c conda-forge molmod
or alternativelyy conda install -c molmod tamkin nose
it keeps giving me errors like:
Do you have an idea what is going wrong here?
Best Regards
Wassja
The text was updated successfully, but these errors were encountered: