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

WIP: introduce Visualizer for rendering in notebooks (#247) - revisited. #469

Closed
wants to merge 1 commit into from

Conversation

liammagee
Copy link

Updates #247 in the following ways:

  • Renames ipython.py to JupyterVisualizer.py, and adds some ability to handle dependencies like Chart.js
  • Includes a Jupyter notebook example, with a working chart
  • Adds a ipywidget slider for fps

Still POC, and willing to help develop further.

@jackiekazil
Copy link
Member

This is really exciting and something that has sat on the back burner for awhile after @smacleod's experiment. Adding Steven to the convo, just in case there was some great insight that he had stored in the back of his head.

@jackiekazil
Copy link
Member

Hello @liammagee -- It looks like there isn't movement on this in awhile. I really hope you have a chance to finish on this. Looking at the Travis integration, it looks like it failed on this...
https://github.com/projectmesa/mesa/blob/master/CONTRIBUTING.rst#testing-and-code-standards
^^ You can run those commands locally to help you work through those.

For now, I am going to close this PR and track work in ticket #246. Please resubmit when complete OR when you want another pair of eyes again. I really would love to see this added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants