IEEERobotics / high-level

CV, localization, mapping, planning, and generally anything that will run on the PandaBoard
BSD 2-Clause "Simplified" License
2 stars 1 forks source link

No goToBlock function in Planner.py #24

Closed dfarrell07 closed 11 years ago

dfarrell07 commented 11 years ago

I'm trying to do full-system tests but am experiencing more bugs in Planner.py. I fixed a few simple ones, but this one's non-obvious.

In Planner.py's processSeaLand function, it makes a call to self.goToBlock, like this:

self.goToBlock(block.getLocation())             

However, there's no goToBlock function in our repo. I don't know how to fix this because it's a logic error, not a syntax error. This bug can be recreated by running controller.py with a freshly cloned repo.

R00ney commented 11 years ago

If there's no function, does that mean it's not been created yet, or it just isn't linked right?

Neal

On Mon, Apr 1, 2013 at 2:57 AM, Daniel Farrell notifications@github.comwrote:

I'm trying to do full-system tests but am experiencing more bugs in Planner.py. I fixed a few simple ones, but this one's non-obvious.

In Planner.py's processSeaLand function, it makes a call to self.goToBlock, like this:

self.goToBlock(block.getLocation())

However, there's no goToBlock function in our repo. I don't know how to fix this because it's a logic error, not a syntax error. This bug can be recreated by running controller.py with a freshly cloned repo.

— Reply to this email directly or view it on GitHubhttps://github.com/NCSUhardware/high-level/issues/24 .

dfarrell07 commented 11 years ago

It hasn't been created:

daniel@ud:~/b/qwe$ grep -rni "goToBlock" ./*
./planning/Planner.py:119:      self.goToBlock(block.getLocation())             
daniel@ud:~/b/qwe$ 
ghost commented 11 years ago

if there's no function, it means it is not required, at least in this case.