-
Notifications
You must be signed in to change notification settings - Fork 192
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
Scan, dock and deliver practice worlds #591
Conversation
Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
@M1chaelM , the environmental conditions have been updated. This PR is ready for review. |
Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
I cannot fire the ball shooter using the "A" button. Is this a known deficiency? |
Color sequences and placards look good, and I'm able to score points docking the correct bay and reporting the correct color sequence. I'm not sure how to fire the ball shooter. Running:
returns the following repeated message:
Looking through the output of
but there does not seem to be any analogous topic listed in the output of |
Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
I think I fixed the ball shooter. The bridge wasn't properly created. You can launch the ball now with:
|
I think I know what's going on with the keypad, I'll fix it after the release. |
Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
Never mind, I also restored the ability to launch the ball with the keypad. If you do it programmatically, it should now be:
|
Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
I found a typo in the activation area of the big hole in placard |
OK, I think it's working, with the caveat that it seems like there's often 1 hit that doesn't get counted. Here's how I'm testing:
Once when I did this I hit the target 4 times but only got credit for 3. The first hit was ignored. I'm not sure whether this problem could be created by the fact that I'm pausing, teleporting and unpausing. I tried a second time and moved the WAMV around a little bit before firing. In the second run my first hit was scored, but the second hit was ignored. If you have any ideas about why this is happening and / or a quick fix, then I can definitely test it again before we merge. Otherwise, since there's a possibility that the problem would not arise in normal use, maybe we should just document the issue and then go ahead with the merge. |
No ideas right now and I don´t think it's related with the pause/unpause part. |
Do you want to keep working on it? Or write up an issue or merge? I'm ok with either. |
I don´t see the issue right now, I'm leaning towards merging it and continue looking at it after the initial relase. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, sounds good.
How to test it?
You can launch the three practice worlds with:
Verify that the robotx color sequences are
red-green-yellow
,yellow-blue-red
andyellow-red-green
respectively.Verify that you can see a
red_rectangle
,yellow_circle
and ayellow_triangle
respectively in one of the placards. The other placard symbols are not that relevant.