PyroTeam / robocup-pkg

7 stars 1 forks source link

[executeur_taches][navigation_manager][path_tracker] Bug possible #32

Open ValentinVERGEZ opened 8 years ago

ValentinVERGEZ commented 8 years ago

Sur un ordre d'exploration à l’exécuteur de tâches, l’exécuteur a semble-t-il demandé plusieurs déplacements (au moins deux). Il a demandé le deuxième avant que le premier finisse et le robot a d'abord terminé son premier déplacement avant d'attaquer le deuxième.

Problème : le second déplacement ayant été demandé quasiment en même temps que le premier, il avait le même point de départ. Résultat, quand le robot a attaqué son second déplacement, il était très loin du chemin à suivre.

Soit le robot aurait du interrompre tout de suite son premier déplacement pour commencer le second, soit le second aurait dû être demandé après la fin du premier (pour que le chemin généré parte bien de la position du robot).

Bref, à surveiller.

tdanel commented 8 years ago

Pas vu depuis

ValentinVERGEZ commented 8 years ago

Descend le timeout de déplacement à 5secs, c'est plus probable que tu le vois ainsi (si toujours présent)

tdanel commented 8 years ago

Sauf que les chemins ne sont plus bufferisés là donc yaura plus le pb, nan ?

ValentinVERGEZ commented 8 years ago

Dans le pathtracker tu veux dire ? Oui, si c'était le problème et qu'on ne passe pas à côté d'autre chose.

Je veux bien fermer l'issue, au regard du comportement du robot sur des ordres au clic, il repart bien de sa position.

Ceci dit, est-ce que c'est normal que navigation-manager empile les ordres plutôt que de les refuser ? A moins que ce soit le noeud de "clic clic" qui les empile...

Le mar. 14 juin 2016 08:35, Thomas DANEL notifications@github.com a écrit :

Sauf que les chemins ne sont plus bufferisés là donc yaura plus le pb, nan ?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/PyroTeam/robocup-pkg/issues/32#issuecomment-225791941, or mute the thread https://github.com/notifications/unsubscribe/AGdpbvHiYE1kiyn8NwWB6OLsH-J4eHWSks5qLkuRgaJpZM4G4GHZ .

vcoelen commented 8 years ago

Pourquoi fermer? Est-ce qu'on est sûr que le problème est réglé. Notamment, j'ai toujours un empilement des ordres au clic. (Éventuellement, on peut ouvrir une issue pour ça, puisque ça n'est pas tout à fait le même problème que l'issue initiale)

tdanel commented 8 years ago

Faux problème. On ne va pas cliquer pendant un match ni en simulation.

vcoelen commented 8 years ago

Ben ça dépend, c'est juste une question, mais si ça arrive au clic, est-ce qu'on a vraiment aucun risque que ça arrive dans une autre situation?

ValentinVERGEZ commented 8 years ago

Oui, c'est pas exclu que ce soit le nœud python (moveRequest_node) qui accumule les demandes, mais si ce n'est pas le cas alors c'est plus grave.

C'est assez facile à déterminer, mais il faut le confirmer avant de fermer cette issue svp. Je m'ajoute aux assignee pour ce qui est de déterminer si c'est un vrai problème ou un faux.