Skip to content

debug_drawer expansion into strategy/position files#2519

Merged
CameronLyon merged 8 commits into
ros2from
debug-drawer-expansion
Apr 14, 2026
Merged

debug_drawer expansion into strategy/position files#2519
CameronLyon merged 8 commits into
ros2from
debug-drawer-expansion

Conversation

@CameronLyon
Copy link
Copy Markdown
Contributor

This addition to the stack is not necessary, but I believe will help tremendously towards debugging and adding additional visual features to the simulator

Implementing debug_drawer among all instances of position/strategy for the sim:

  • Position classes like offense.cpp, defence.cpp, seeker.cpp, solo_offense.cpp, and many more will now be capable of incorporating debug_drawer features (drawing circles/squares, inputting text, etc)
  • This feature is linked to a flag variable (located in position.hpp called "debug_draw_enabled_") which is defaulted to false, but can be selectively enabled for specific position roles
  • Example of enabling and incorporating this debugging feature is shown in offense.cpp, where offense bots will now showcase their current state when moving, avoiding the messy need to utilize SPDLOG_INFO and flood the terminal with print statements. If you want to remove this, just let me know with a comment and I can comment out the currently enabled example

automated style fixes

Co-authored-by: CameronLyon <CameronLyon@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@Squid5678 Squid5678 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

automated style fixes

Co-authored-by: shourikb <shourikb@users.noreply.github.com>
@CameronLyon CameronLyon merged commit 7bfeb7d into ros2 Apr 14, 2026
2 checks passed
@CameronLyon CameronLyon deleted the debug-drawer-expansion branch April 14, 2026 23:56
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

Successfully merging this pull request may close these issues.

4 participants