ProgrammingRobotsStudyGroup / robo_magellan

Apache License 2.0
4 stars 6 forks source link

Minor changes to wheel encoder and touch sensor testing #22

Closed jeremyolsen closed 6 years ago

jeremyolsen commented 6 years ago

Found changes during testing.

The majority of the changes were just setting defaults on the incoming parameters. I was debugging via PyCharms and the params from the launch were not going to be loaded in this scenario, thus the reason for adding them.

The 's' variable needed to be initialized outside the if statement. I got the error while launching in ROS.

merose commented 6 years ago

I think the node name and type are misspelled. Should be "arduino", not "arudino". Will fix after pulling.