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

Add graph visualization tool. #189

Merged
merged 4 commits into from
Feb 13, 2020

Conversation

MarcCote
Copy link
Contributor

@MarcCote MarcCote commented Oct 12, 2019

This PR adds support for visualizing the underlying graph of a game state.
image

This PR also fixes #162

TODO:

  • Make visualization dependencies optional.
  • Add test for show_graph.

@codecov-io
Copy link

codecov-io commented Nov 29, 2019

Codecov Report

Merging #189 into master will decrease coverage by <.01%.
The diff coverage is 78.06%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #189      +/-   ##
==========================================
- Coverage   81.01%   81.01%   -0.01%     
==========================================
  Files          56       57       +1     
  Lines        6757     6925     +168     
==========================================
+ Hits         5474     5610     +136     
- Misses       1283     1315      +32
Impacted Files Coverage Δ
textworld/render/__init__.py 100% <100%> (ø) ⬆️
textworld/envs/wrappers/viewer.py 80.95% <50%> (+0.46%) ⬆️
textworld/render/serve.py 47.64% <63.88%> (-0.36%) ⬇️
textworld/render/render.py 60.37% <69.56%> (+3.78%) ⬆️
textworld/render/graph.py 91.66% <91.66%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a820f38...d99df57. Read the comment docs.

@MarcCote MarcCote force-pushed the enh_visualize_graph branch 2 times, most recently from e8088ed to e682182 Compare February 12, 2020 19:48
@MarcCote MarcCote force-pushed the enh_visualize_graph branch 2 times, most recently from dca4252 to b41135c Compare February 13, 2020 01:34
@MarcCote MarcCote merged commit 51225ec into microsoft:master Feb 13, 2020
@MarcCote MarcCote deleted the enh_visualize_graph branch February 13, 2020 03:01
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.

Make prompt-toolkit an optional dependency.
2 participants