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

Highlight current state in Graphviz visualizer #84

Merged
merged 1 commit into from
Jul 15, 2022

Conversation

baw-benji
Copy link
Contributor

@baw-benji baw-benji commented May 4, 2022

This PR changes the current Graphviz visualizer in the following way:

  • Keep the drawing order when current state changes to have a stable state diagram output.
    Instead, the current state is highlighted.

@coveralls
Copy link

coveralls commented May 7, 2022

Coverage Status

Coverage decreased (-0.08%) to 93.573% when pulling f5c8c9d on baw-benji:graphviz into 0b86adf on looplab:main.

We want to have a stable graph output.

Signed-off-by: Benjamin Imhof <benjamin.imhof@bytesatwork.ch>
@baw-benji
Copy link
Contributor Author

Coverage Status

Coverage decreased (-0.08%) to 93.59% when pulling 590c2c5 on baw-benji:graphviz into 72af45e on looplab:main.

All relevant lines of code are covered in graphviz_visualizer.go. What can I test more?

@baw-benji baw-benji requested a review from maxekman July 13, 2022 15:25
@maxekman maxekman merged commit 13de56e into looplab:main Jul 15, 2022
@maxekman
Copy link
Member

👍

@baw-benji baw-benji deleted the graphviz branch July 25, 2022 14:27
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

Successfully merging this pull request may close these issues.

3 participants