-
Notifications
You must be signed in to change notification settings - Fork 6
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
Petri graph layout clean new branch #391
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added tests.
Changed variable names in the graph schema to clarify their role.
Removed a toggle in the schema whose action was hard to understand and whose purpose was difficult to imagine beyond testing.
@JosephCottam and @marjoleinpnnl , I'm going to revert this PR and restore the branch because of the conflicts with the major refactor going on in |
Merges jc-graph-refinments with Petri graph with fixed layout with update so it keeps all moved nodes in a fixed position and double clicking unfixes the node. Petri graph is viewable in notebook/visual examples/Graphs.ipynb