Skip to content

Improve planners architecture#16

Merged
guilyx merged 13 commits intomainfrom
add-dijktra
Dec 20, 2020
Merged

Improve planners architecture#16
guilyx merged 13 commits intomainfrom
add-dijktra

Conversation

@guilyx
Copy link
Copy Markdown
Member

@guilyx guilyx commented Dec 20, 2020

  • It's now easier to understand the planners, one planner has a file and a separate class that takes the AutomatedPlanning main class as argument
  • Tested the new BFS and initialized empty Dijkstra/A*/DFS

@guilyx guilyx added enhancement New feature or request architecture labels Dec 20, 2020
@guilyx guilyx added this to the Minimum Product milestone Dec 20, 2020
@guilyx guilyx requested a review from sampreets3 December 20, 2020 13:40
@guilyx guilyx mentioned this pull request Dec 20, 2020
@guilyx guilyx removed the request for review from sampreets3 December 20, 2020 17:47
@guilyx guilyx merged commit ad50fa5 into main Dec 20, 2020
@guilyx guilyx deleted the add-dijktra branch December 21, 2020 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

architecture enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant