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

New plots for the network example #6

Closed
10 of 11 tasks
riberaborrell opened this issue Nov 15, 2019 · 4 comments
Closed
10 of 11 tasks

New plots for the network example #6

riberaborrell opened this issue Nov 15, 2019 · 4 comments
Assignees

Comments

@riberaborrell
Copy link
Collaborator

riberaborrell commented Nov 15, 2019

re-write

  • plot_density() method
  • plot_effective_current() method
    To do:
  • In the ergodic case: remove edge 1-2 from visualization since the transition probability between 1 and 2 is zero.
  • Similarly, remove edges from the periodic and finite-time case when zero transition probability.
  • Color bar in the right
  • The normalized reactive density is not defined for times n=0 and n=N-1 for the finite-time case.
  • similar for the current
  • similar for the rate
  • for plotting the rates: use plt.ylim(np.nanmin(rate_p)-0.005, np.nanmax(rate_p)+0.005) to change the y limits of the rate plot and use plt.legend(ncol=2) to show the two (or three) legend items next to another
  • different colorscheme such that the numbers on the nodes (e.g. A, B,2,...) can be read well (also Niklas was confused that the color of the reactive density in the network example is purple in A and B, maybe its more intuitive/better to have a color scheme that gives color white to the value 0)
  • increase the image quality
@riberaborrell riberaborrell self-assigned this Nov 15, 2019
@riberaborrell
Copy link
Collaborator Author

Luzie's suggestion:
charts_sketch_small_network

@riberaborrell
Copy link
Collaborator Author

networkx allows to have multiple edges but does not plot them as we wish:
https://stackoverflow.com/questions/22785849/drawing-multiple-edges-between-two-nodes-with-networkx

Repository owner deleted a comment from LuzieH Nov 21, 2019
riberaborrell pushed a commit that referenced this issue Nov 22, 2019
Repository owner deleted a comment from LuzieH Nov 27, 2019
riberaborrell pushed a commit that referenced this issue Nov 27, 2019
riberaborrell pushed a commit that referenced this issue Nov 27, 2019
riberaborrell pushed a commit that referenced this issue Nov 28, 2019
riberaborrell pushed a commit that referenced this issue Nov 29, 2019
riberaborrell pushed a commit that referenced this issue Nov 29, 2019
@LuzieH LuzieH added this to the Submission of the Paper milestone Dec 18, 2019
@LuzieH
Copy link
Owner

LuzieH commented Dec 18, 2019

There is some problem with the plots of the small network.

@LuzieH
Copy link
Owner

LuzieH commented Dec 18, 2019

small_network_reac_dens_inhom

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