srvk / DiViMe

ACLEW Diarization Virtual Machine
Apache License 2.0
32 stars 9 forks source link

Numpy / Theano version problem? Noisemes fails selftest #109

Closed riebling closed 5 years ago

riebling commented 5 years ago

Something must have changed I have not yet managed to track down, causing Noisemes to fail the self-test. (may be a numpy / theano update problem) Working backwards from things I've tried:

So, something is funky with versions of Python packages related to Theano and numpy

riebling commented 5 years ago

To verify the steps I took to 'fix' this, I tried from a fresh 'vagrant up', the test passed as above. Then these steps seemed to 'fix' the issue:

  155  rm -rf ~/anaconda
  156  ls
  157  wget -q https://3230d63b5fc54e62148e-c95ac804525aac4b6dba79b00b39d1d3.ssl.cf1.rackcdn.com/Anaconda-2.3.0-Linux-\
x86_64.sh
  158  bash Anaconda-2.3.0-Linux-x86_64.sh -b
  159  conda install numpy scipy mkl dill tabulate joblib cython=0.22.1 sphinx
  160  launcher/test.sh

What is INTERESTING to observe: although bootstrap.sh installs explicitly theano=0.8.2, after my brute force reinstall, the output of pip freeze | grep -i theano (is this even the right way to check Python package versions?!) shows a different Theano version:

numpy==1.10.2
vagrant@vagrant-ubuntu-trusty-64:~$ pip freeze | grep -i theano
You are using pip version 9.0.1, however version 19.0.1 is available.                                                  
You should consider upgrading via the 'pip install --upgrade pip' command.
Theano==0.7.0

I'm worried about hard coding / forcing Theano version just so Noisemes passes it's test, if it results in degraded/undesirable performance of other tools that use it: where did the hard dependency of theano=0.8.2 come from? (probably Yun)

riebling commented 5 years ago

Sure enough, now Diartk and VCM fail their selftests with Theano==0.7.0. Not the right solution :( I defer to people who are more Python users/experts to help figure out: is it OK the way we're mixing the use of pip install and conda install? Need to better understand this. It almost seems like bootstrap.sh may have swapped some of these.

Whoa, installing just theano=0.8.2 with conda, now Noismes, Diartk, Yunitator, VCM, WCE - ALL fail. Definitely not a 'fix'!

riebling commented 5 years ago

Ugh. This was for some reason running the major.reorganization branch. I swear I checked out main, I mean head, I mean master, when I started things. Hope that wasn't a problem as well.

MarvinLvn commented 5 years ago

I've seen that the major_reorganization branch had 3 commits ahead master (here) Could you integrate these modifications to the master branch ? That way I'll delete this old and fragile branch.

riebling commented 5 years ago

Ok, done! I'm not sure how my working directory ended up on major_reorganization. Good to be in sync again.