rosjava / android_remocons

Android based remote controllers for 1-1 robot pairing and multi-robot concerts.
21 stars 28 forks source link

robot_remocon crashes when pairing master not fully ready #23

Closed bit-pirate closed 11 years ago

bit-pirate commented 11 years ago

There is some delay between the paring master (public) is available (robot_remocon finds it) and it being fully configured/ready. I currently believe this is the time interval when the pushing and pulling is going on. If the robot_remocon connects during that time it crashes 100% of the time.

stonier commented 11 years ago

Ah, good point. I need a wait for service of some sort. On Jul 3, 2013 9:44 AM, "Marcus Liebhardt" notifications@github.com wrote:

There is some delay between the paring master (public) is available (robot_remocon finds it) and it being fully configured/ready. I currently believe this is the time interval when the pushing and pulling is going on. If the robot_remocon connects during that time it crashes 100% of the time.

— Reply to this email directly or view it on GitHubhttps://github.com/ros-java/android_remocons/issues/23 .

stonier commented 11 years ago

https://github.com/ros-java/android_apps/commit/7269f2798c350917bb4e8695078368f409acb18f

It wasn't checking to see if it had actually found the app manager namespace (via master system state lookups), and thus crashed when the namespace was still null.