Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Waypoints with invalid pose #3

Open
jim-v opened this issue May 9, 2017 · 7 comments
Open

Waypoints with invalid pose #3

jim-v opened this issue May 9, 2017 · 7 comments
Assignees

Comments

@jim-v
Copy link

jim-v commented May 9, 2017

If a waypoint is set when the robot location is unknown, then no-error is reported, but when asked to go to that waypoint, RobotCommander sends the following goal:

header:
seq: 16
stamp:
secs: 0
nsecs: 0
frame_id: ''
goal_id:
stamp:
secs: 0
nsecs: 0
id: goal_0.06402862432775747_1494305466006
goal:
target_pose:
header:
seq: 0
stamp:
secs: 1494305466
nsecs: 619306087
frame_id: /map
pose:
position:
x: 0.0
y: 0.0
z: 0.0
orientation:
x: 0.0
y: 0.0
z: 0.0
w: 0.0

@jim-v jim-v assigned jim-v and jrlandau and unassigned jim-v May 9, 2017
@jrlandau
Copy link
Collaborator

jrlandau commented May 9, 2017 via email

@jim-v
Copy link
Author

jim-v commented May 9, 2017

There should be an exception thrown if the current pose can't be determined. You should be able to catch that and give the user an error message that the waypoint can't be saved because the robot's position is not known.
x=0=y=0 is a valid position, but the quaternion with all elements set to 0 is not valid.

@rohbotics
Copy link
Member

I think If there is no pose data available, it shoudn't default to 0s, it should error when setting the waypoint

Regardless, It shouldn't send an invalid goal
The orientation is a quaternion and

x: 0.0
y: 0.0
z: 0.0
w: 0.0

is not a valid quaternion
The valid quaterion for zero is

x: 0.0
y: 0.0
z: 0.0
w: 1.0

@jrlandau
Copy link
Collaborator

jrlandau commented May 9, 2017 via email

@davecrawley
Copy link

Agree, invalid pose should draw human attention to the error. (Poka-yoke)

@jim-v
Copy link
Author

jim-v commented May 9, 2017

On a robot, don't run localization, or cover the camera, or turn the lights off, then set a waypoint.

@jrlandau
Copy link
Collaborator

If an invalid pose is received, the error is caught and a response is given to the user informing her. If a pose is requested and it is never received, the failure is silent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants