RussTedrake / manipulation

Course notes for MIT manipulation class
BSD 3-Clause "New" or "Revised" License
407 stars 119 forks source link

Bug in AddCameraBox implementation? #272

Closed sizhe-li closed 9 months ago

sizhe-li commented 9 months ago

Hi Russ and course staff,

I think the AddCameraBox function has bugs.

In L280, parser = Plant(parser) should instead be parser=Parser(plant)?

Additionally, L288 should have [0] indexing at the end because AddModelsFromString currently returns a list of length one.

Sizhe

RussTedrake commented 9 months ago

Indeed. Clearly, it's not being used. I will zap it.