-
Notifications
You must be signed in to change notification settings - Fork 198
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 Failure #86
Comments
Hi @davidgwyrick , my name is Ben -- I'm working on SSM with Scott. Thanks for reporting this, someone else actually mentioned this to me before, but I wasn't able to reproduce it on my own machine. It seems like it was caused by switching messages.py to use Numba, rather than Cython, but I don't know exactly why that is causing this. This is a good reminder to make sure we get this fixed. Could you try this and let me know the result: create a new clean virtualenv with pip (not a conda environment) which doesn't have SSM installed, and try installing there? The steps should look something like this: Let me know if that resolves the error. |
Hey Ben, thanks for getting back to me. Now it looks like there are other syntax errors that cause the nosetests to fail. `(test_env) dwyrick:ssm$ nosetests
|
Hi David -- I think these new errors are probably a python 2.X vs. python 3.X issue. In the output, I see some /usr/lib/python2.7/ which I think means you are running Python 2.7 (although it might be possible that you're running Python 3 but somehow the old packages are being called). Since there are some syntax differences between Python 2 and 3, this might explain some of what you're seeing. We only support and test with Python 3 (although we certainly should be more explicit about this in the documentation.) Can you check if you're using Python 3 (preferably 3.7) and let me know if that fixes things? |
Weird, I had created the virtual environment with the python3.7 option, but I guess nosetests was still pointing to the python2.7 package. I don't usually use pip for my package manager, but I think I got things to work.. although when I run the nosetests, i do get the following UserWarnings and the following new* error.
`======================================================================
|
Okay great, this looks like progress. The user warnings are nothing to worry about -- they come up because we sometimes have to take the log of zero. Hopefully we'll be able to squash them to clean things up in a future commit. I haven't seen the divide by zero issue happen in dirichlet.py. I'll have to see if I can reproduce that. Just to double check, this is on the latest branch of master? |
Indeed, this is the latest branch of master. It also looks like the notebook examples are running fine. |
Hey Scott,
I'm getting 4 errors on the nosetests after doing a fresh install of ssm in a new conda environment. From the looks of it, they all seem to be related to an import error of 'hmm_expected_states'. Is this related to cython or something, since messages.py is cythonized? I followed your instructions on the install and tried a few variations but to no avail. Any ideas?
Cheers,
David
The text was updated successfully, but these errors were encountered: