simulator icon indicating copy to clipboard operation
simulator copied to clipboard

potential bug in NPCController.SetChangeLaneData()

Open david-gwa opened this issue 5 years ago • 1 comments

hey, Lg-team,

for lane change scenario, with a different map, where the wps indexes are not consistent in neighbor lanes, there is a problem to use currentIndex from previous lane to update currentTarget for current lane (line891 in NPController) :

   private void SetChangeLaneData(List<Vector3> data)
   {
       laneData = new List<Vector3>(data);
       currentTarget = laneData[currentIndex];  
       isDodge = false; // ???
   }

we are also looking for a more robost algorithms to set new target in new lane

david-gwa avatar Sep 29 '19 05:09 david-gwa

@david-gwa

Yes, you are correct. We are adding some new features to NPCs and will look at a fix for this. Thanks for letting us know.

EricBoiseLGSVL avatar Sep 30 '19 18:09 EricBoiseLGSVL