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

graph should not write always an image #96

Closed
spaceone opened this issue Jun 13, 2015 · 6 comments · Fixed by #303
Closed

graph should not write always an image #96

spaceone opened this issue Jun 13, 2015 · 6 comments · Fixed by #303
Assignees
Milestone

Comments

@spaceone
Copy link
Contributor

We should split the graph() function. One function should write the image file (where one can specify also a filename/path). The other one should return the ascii-art. Both use cases exists. Both functions are very nice!

@prologic
Copy link
Member

Actually... I'm thinking we should remove the image functionality and leave that to external tools. We should improve the "graphing" API(s) such that we have:

  • graph() ASCII graph representation of the system
  • dotgraph() DOT graph representation of the system
  • ... Something that some kind of real-time JS graphing frontend can render

@ri0t
Copy link
Contributor

ri0t commented Jun 13, 2015

Check this one out: http://cytoscape.github.io/cytoscape.js/demos/5b192c88616af2f75344/
I like it. Plays well with parent/child relations and can visualize handlers and possibly with few additional code even live events.
The library's home is here: http://cytoscape.github.io/cytoscape.js/

@prologic
Copy link
Member

We should build a mini webapp with an API that drives data for real-time gendering of the system component grpah:

Maybe using this library: http://cytoscape.github.io/cytoscape.js/

Demo: http://cytoscape.github.io/cytoscape.js/demos/5b192c88616af2f75344/

@prologic
Copy link
Member

@riot haha jinx; )

@prologic prologic modified the milestone: 3.3 Aug 13, 2015
spaceone added a commit that referenced this issue Jan 24, 2017
@spaceone
Copy link
Contributor Author

@prologic
ec9f5b9
I don't know it it's possible to let the image e.g. be written into a io.BytesIO() ? Then one could give a file descriptor as argument.

@prologic
Copy link
Member

Yeap you can use:

buf = io.BytesIO()
plt.savefig(buf, format='png')

@spaceone spaceone self-assigned this Jan 30, 2017
spaceone added a commit that referenced this issue Aug 20, 2021
spaceone added a commit that referenced this issue Aug 20, 2021
spaceone added a commit that referenced this issue Oct 19, 2021
spaceone added a commit that referenced this issue Oct 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants