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.

Sign up for GitHub

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

[Garden] Port scan, dock, deliver - Part 1 #537

Merged
merged 7 commits into from
Nov 18, 2022
Merged

[Garden] Port scan, dock, deliver - Part 1 #537

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
df5e229
Initial layout.
caguero Nov 1, 2022
25f2318
Part 1.
caguero Nov 4, 2022
f942599
Tweaks
caguero Nov 9, 2022
2c2645b
Tweak
caguero Nov 9, 2022
c42a357
Merge from gazebosim
caguero Nov 9, 2022
e54e92d
Remove unused bridges.
caguero Nov 9, 2022
558ef4d
Only exactly three colors are allowed.
caguero Nov 18, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Only exactly three colors are allowed. 
Signed-off-by: Carlos Agüero <caguero@openrobotics.org>
  • Loading branch information
@caguero
caguero committed Nov 18, 2022
commit 558ef4d29f251d7756f965fd462c5404cc86305c
2 changes: 1 addition & 1 deletion vrx_gz/src/ScanDockScoringPlugin.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ void ColorSequenceChecker::OnColorSequence(const msgs::StringMsg_V &_sequence)
}

// Sanity check: We should receive three colors.
if (_sequence.data_size() < 3u)
if (_sequence.data_size() != 3u)
{
gzerr << "Received a color sequence with " << _sequence.data_size()
<< " values but expecting three instead. Submission ignored."
Expand Down