markovmodel / PyEMMA

🚂 Python API for Emma's Markov Model Algorithms 🚂
http://pyemma.org
GNU Lesser General Public License v3.0
311 stars 119 forks source link

[plots] plot_state_map #1335

Closed thempel closed 6 years ago

thempel commented 6 years ago

It seems that the plot_state_map() doesn't work properly, cf. the notebook example in pyemma_tutorials, notebook 05, 6 state HMM. Where's the sixth state in the plot below? It's definitely in the data...

index

Something is wrong with the colormap, I guess. State zero is not distinguished from state 1.

Further, I don't quite understand why the plot_state_map() function needs the nstates keyword argument. If I'm not mistaken, I deleted it some time ago and it was added again. A state map should be defined by the number of states in the list of labels that is passed to this function, right? Why should a user want to have a different number?

tagging @cwehmeyer

cwehmeyer commented 6 years ago

Are you using the latest devel build? There has been an issue with a missing ncountours parameter in plot_state_map which has been resolved in #1331.

thempel commented 6 years ago

Ok, works with devel. Sorry.