Skip to content

final push pathfinder+astar+spfa#30

Open
joffrey3 wants to merge 1 commit intobcalou:mainfrom
joffrey3:main
Open

final push pathfinder+astar+spfa#30
joffrey3 wants to merge 1 commit intobcalou:mainfrom
joffrey3:main

Conversation

@joffrey3
Copy link
Copy Markdown

Bonjour Monsieur,
Voici le TP que vous nous avez demandé pour le 18 février Désolé pour le retard occasioné
Cordialement, Joffrey Fouché

Bonjour Monsieur,
Voici le TP que vous nous avez demandé pour le 18 février
Désolé pour le retard occasioné
Cordialement, Joffrey Fouché
@joffrey3 joffrey3 closed this Feb 18, 2024
@joffrey3 joffrey3 reopened this Feb 18, 2024
@joffrey3
Copy link
Copy Markdown
Author

Rendu Fouché Joffrey

@bcalou
Copy link
Copy Markdown
Owner

bcalou commented Feb 26, 2024

Bon travail (pas en retard, c'était dans le temps je crois !).
Soin apporté au code. Par contre beaucoup de redondance entre les trois classes, on pouvait mieux tirer parti de l'héritage (voir correction).
Le test AStar ne passe pas. L'algo doit être identique à Djiestra, à part deux points :

  • usage de l'heuristique pour choisir la prochaine ville
  • l'algo s'arrête dès qu'une solution est trouvée, même si ce n'est pas forcément la meilleure

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