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

plotFluxPathways not plotting #246

Closed
j3mdamas opened this issue Feb 1, 2017 · 4 comments
Closed

plotFluxPathways not plotting #246

j3mdamas opened this issue Feb 1, 2017 · 4 comments

Comments

@j3mdamas
Copy link
Contributor

j3mdamas commented Feb 1, 2017

pyemma was updated to 2.3, maybe our call it outdated?

I noticed that the problem was in pyemma 2.2. Reported here markovmodel/PyEMMA#1032. I was using PyEMMA tutorial to test the plot_flux command we call with plotFluxPathways. With v2.2 it didn't work, but cloning the repo and using the latest, it works (the tutorial with plot_flux).

Going back to our plotFluxPathways, now using the latest pyemma (which by the way isn't on omnia, markovmodel/PyEMMA#1033), I get no plot and the following:

/home/joao/maindisk/software/repos/markovmodel/PyEMMA/build/lib.linux-x86_64-3.5/pyemma/plots/networks.py:544: FutureWarning: elementwise comparison failed; returning scalar instead, but in the future will perform elementwise comparison
  if state_labels == 'auto':

I am guessing that the new plot_flux does not like the state_labels we are passing? Changing it to 'auto' gets rid of the Warning, but still no plot.

@stefdoerr, maybe the mapping of the sets is failing? I'm really surprised the plot does not show but there is no error or traceback...

@stefdoerr
Copy link
Contributor

At 2.2.7 it works fine for me.

@j3mdamas
Copy link
Contributor Author

j3mdamas commented Feb 2, 2017

could you think of any reason why it does not work for me? do I need a fresh conda installation, maybe?

@stefdoerr
Copy link
Contributor

stefdoerr commented Feb 2, 2017

Works fine in 2.3 as well.
I will need a test case that can be replicated.

@j3mdamas
Copy link
Contributor Author

j3mdamas commented Feb 2, 2017 via email

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

2 participants