Skip to content

Conversation

@Rishav159
Copy link
Contributor

@Rishav159 Rishav159 commented Aug 5, 2017

This PR does the following:

  • Fixes the color inconsistencies among diagrams and legends.
  • Makes the nodes in the bfs, dfs and step costs diagram circular.
  • Removes edge labels from depth limited search and iterative depth limited search diagrams.
  • Switches the positions of standard bfs and bi-directional bfs to have a 'before-after' comparison
  • Switches the positions of bfs shortest path and lowest cost path in the step costs diagram for the same reason
  • Changes the node count label of bi-directional bfs diagram from x to x nodes
  • Fixes the bug of node count going up-down if the diagram is restarted quickly.
  • Adds a hover effect to Node-Expansion diagrams to indicate the node is clickable and also to easily compare the frontier nodes with frontier list on the right.

Improvement on #57

@redblobgames
Copy link
Contributor

Hmm, the lighter color for breadth first search looks better in the small graph with large circles than in the large graph with small circles. I guess this is one of those cases where the same color can look a little different based on the surroundings. We can tweak it again later.

@redblobgames redblobgames merged commit afd64ca into aimacode:master Aug 5, 2017
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.

2 participants