-
Notifications
You must be signed in to change notification settings - Fork 363
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
Can't run notebook example #8
Comments
Hmm.. odd that the travis tests didn't detect this. What version of pandas are you using? |
0.13.1 On 06/16, Ben Mabey wrote:
|
@ariddell okay, a newer version of pandas may help. For now though could you clone master of pyLDAVis and install it ( |
I'm not having much luck getting the newer IPython installed, so this is proving hard to test. I'll keep you posted if I figure it out. |
To test this fix I made you don't need any newer libraries aside from installing this one from master. |
Nothing's showing up in my old IPython notebook window although the error does disappear (there are just a lot of numpy warnings). Sorry for the imprecise report. I can open a new issue if I have time to dig into this and reproduce it. |
Could there be a circular import somewhere? Now I'm getting this:
|
I have seen that problem in the past but it has happened when I reload the modules during development. I didn't think it would happen with clean/initial imports, at least I've never seen it happen in that case. I don't see a circular dep but I could be wrong. I'm not sure how else to explain the error since my python experience is limited. |
No worries. I'll get to the bottom of it eventually. Thanks for the project! |
* 'master' of https://github.com/bmabey/pyLDAvis: (84 commits) fixes other typo in gl notebook fix typo use the graphlab API for it's prepare, 3x+ speed improvement! updates notebook with models from LDAvis package replaces use of map with list comprehension to help with bmabey#8 pass kargs to prepare fn in gensim and graphlab helpers. Closes bmabey#6, bmabey#7. releases v1.2.0 adds link to the R vignette that explains the visualization prevents prepare from blowing up when used with a vocab smaller than R make the gensim extraction more explicit... trying to find bug in logic updates gensim adapter to get topic term dists directly /ht @cscorley fixes Makefile to run on osx releases v1.1.0 change how web assets are served from github.. using detatched branch fixes graphlab viz fixes bug in graphlab model prepare causing bogus visualizations! doc fixes fix pypi badge release v1.0.0 prep for v1.0.0 release ...
The library looks great. Thank you!
Just trying to run the notebook example with IPython 3.1.0 and hitting this error (including only the top):
The text was updated successfully, but these errors were encountered: