Nil value detected related to driving skill

[tnguyenh]

Problème de nil value detected, dans l'action 'update', de l'espèce 'car' (vers ligne 1834).

Je pense que c'est en rapport avec l'issue #80 , ce serait bien de régler ça rapidement... je peux réutiliser l'astuce du copy je pense, j'attends de voir si tu as une meilleure idée @ptaillandier

[ptaillandier]

tu peux me dire comment reproduire cette erreur ? Le mar. 11 févr. 2020 à 15:53, Tri Nguyen-Huu <[email protected]> a écrit :

…

Assigned #89 <#89> to @ptaillandier <https://github.com/ptaillandier>. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#89?email_source=notifications&email_token=AALPWHKDD6FBPDTFOVQWPETRCK3W5A5CNFSM4KTCJRA2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWRVFUCQ#event-3026868746>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AALPWHOEVPJV4S2CWYXJKPLRCK3W5ANCNFSM4KTCJRAQ> .

[ptaillandier]

Déjà, je sais d'où vient cette action, mais c'est une sacrée distorsions du driving skill... je ne sais même pas comment elle peut marcher. c'est quoi le bug de cette action ? Le mar. 11 févr. 2020 à 16:08, Patrick Taillandier < [email protected]> a écrit :

…

tu peux me dire comment reproduire cette erreur ? Le mar. 11 févr. 2020 à 15:53, Tri Nguyen-Huu ***@***.***> a écrit : > Assigned #89 <#89> to > @ptaillandier <https://github.com/ptaillandier>. > > — > You are receiving this because you were assigned. > Reply to this email directly, view it on GitHub > <#89?email_source=notifications&email_token=AALPWHKDD6FBPDTFOVQWPETRCK3W5A5CNFSM4KTCJRA2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWRVFUCQ#event-3026868746>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AALPWHOEVPJV4S2CWYXJKPLRCK3W5ANCNFSM4KTCJRAQ> > . >

[ptaillandier]

*je voulais dire le but de cette action. Le mar. 11 févr. 2020 à 16:17, Patrick Taillandier < [email protected]> a écrit :

…

Déjà, je sais d'où vient cette action, mais c'est une sacrée distorsions du driving skill... je ne sais même pas comment elle peut marcher. c'est quoi le bug de cette action ? Le mar. 11 févr. 2020 à 16:08, Patrick Taillandier < ***@***.***> a écrit : > tu peux me dire comment reproduire cette erreur ? > > Le mar. 11 févr. 2020 à 15:53, Tri Nguyen-Huu ***@***.***> > a écrit : > >> Assigned #89 <#89> to >> @ptaillandier <https://github.com/ptaillandier>. >> >> — >> You are receiving this because you were assigned. >> Reply to this email directly, view it on GitHub >> <#89?email_source=notifications&email_token=AALPWHKDD6FBPDTFOVQWPETRCK3W5A5CNFSM4KTCJRA2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWRVFUCQ#event-3026868746>, >> or unsubscribe >> <https://github.com/notifications/unsubscribe-auth/AALPWHOEVPJV4S2CWYXJKPLRCK3W5ANCNFSM4KTCJRAQ> >> . >> >

[tnguyenh]

Désolé réponse tardive, et manque de détails dans l'explication du problème. Mais j'avais réussi à contourner le problème.

Le problème à cette ligne précise est de reconstruire un chemin lorsque le réseau de route est modifié (routes qui passent de n voies à 0). Recalculer un chemin en utilisant les outils existant ne marche pas car les agents cherchent à se repositionner sur un des sommets avant de reprendre leur trajet. Ce qui donne des trajectoires très exotiques.

Du coup le principe était de recalculer un chemin tout en gardant la position actuelle sur la route courante, et donc de calculer l'itinéraire après le sommet suivant, et de recombiner le tout.

Cela marche bien a priori, cependant parfois le code n'est pas suivi exactement: par exemple pour le code suivant:

if current_road != nil{
	road r <- road(current_road);
}

on se retrouve parfois avec une nil value detected au niveau de l'assignation, alors que le test a été passé correctement. J'imagine qu'il y a une histoire de modification concurrente ou de threads concurrent. Du coup en utilisant un road r <- copy(road(current_road)) avant de rentrer dans le statement 'if' ça passe.

[ptaillandier]

Si vous avez pu trouver une solution c'est le principal (j'étais un peu en stress pour vous, mais en même temps, ne pouvant reproduire le bug et de peur de tout casser, je ne pouvais pas faire grand chose). Maintenant que c'est un peu moins le stress, je vais regarder tout ça et éventuellement ajouter quelques nouvelles actions sur le advanced driving skill pour faciliter certaines opérations et le rendre plus robuste. Le mar. 18 févr. 2020 à 01:37, Tri Nguyen-Huu <[email protected]> a écrit :

…

Désolé réponse tardive, et manque de détails dans l'explication du problème. Mais j'avais réussi à contourner le problème. Le problème à cette ligne précise est de reconstruire un chemin lorsque le réseau de route est modifié (routes qui passent de n voies à 0). Recalculer un chemin en utilisant les outils existant ne marche pas car les agents cherchent à se repositionner sur un des sommets avant de reprendre leur trajet. Ce qui donne des trajectoires très exotiques. Du coup le principe était de recalculer un chemin tout en gardant la position actuelle sur la route courante, et donc de calculer l'itinéraire après le sommet suivant, et de recombiner le tout. Cela marche bien a priori, cependant parfois le code n'est pas suivi exactement: par exemple pour le code suivant: road r <- road(current_road); }```` on se retrouve parfois avec une nil value detected au niveau de l'assignation, alors que le test a été passé correctement. J'imagine qu'il y a une histoire de modification concurrente ou de threads concurrent. Du coup en utilisant un road r <- copy(road(current_road)) avant de rentrer dans le statement 'if' ça passe. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#89?email_source=notifications&email_token=AALPWHMLLYTNZATPUQ6QL63RDMUTHA5CNFSM4KTCJRA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMADD6Y#issuecomment-587215355>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AALPWHMNOXIYLDI5BWOQEJTRDMUTHANCNFSM4KTCJRAQ> .