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.

Sign up for GitHub

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

Jump to bottom

Determine pygraphviz from requirements #196

Closed
wtgee opened this issue Dec 16, 2017 · 3 comments
Closed

Determine pygraphviz from requirements #196

wtgee opened this issue Dec 16, 2017 · 3 comments
Labels
good first issue
Milestone
v0.8.0

Comments

@wtgee
Copy link
Member

wtgee commented Dec 16, 2017

It will be removed from requirements.txt as it's not needed and gives install problems. Need to evaluate when/where needed.
@wtgee wtgee mentioned this issue Dec 16, 2017
Merged
@wtgee wtgee mentioned this issue Feb 13, 2018
Closed
@wtgee
Copy link
Member Author

wtgee commented Nov 16, 2018

Note: pytransitions/transitions#325

@wtgee wtgee added this to the v0.8.0 milestone Feb 16, 2020
@wtgee
Copy link
Member Author

wtgee commented Feb 16, 2020

Currently pygraphviz is an optional run-time dependency. What would probably be better is to have a "state machine verification" script or something that will generate these files as a one-off activity rather than something that is done while the unit is actively observing.

Note that pytransitions may have already done the work for this (see Diagrams) section of documentation.

@wtgee
Copy link
Member Author

wtgee commented Jun 1, 2020

Removed all pygraphviz in #951. Can be added back as a feature if needed, but see pytransitions-gui first.

@wtgee wtgee closed this as completed Jun 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue
Projects
None yet
Milestone
v0.8.0
Development

No branches or pull requests
1 participant
@wtgee