Skip to content

Seeker coordinator#2473

Open
jvogt23 wants to merge 13 commits into
ros2from
seeker-coordinator
Open

Seeker coordinator#2473
jvogt23 wants to merge 13 commits into
ros2from
seeker-coordinator

Conversation

@jvogt23

@jvogt23 jvogt23 commented Nov 24, 2025

Copy link
Copy Markdown
Contributor

Description

Added Seeker coordinator to handle seekers and seeker points without requiring several robots to keep track of each other.
Single source of truth for current seeker points.

Associated / Resolved Issue

Resolves # or ClickUp card
Resolves Clickup Issue: 86b6ubv6c

Design Documents

Link

Steps to Test

Test Case 1

  1. Start the sim
  2. Hit play
  3. Seekers go brrrr

Expected result: Seekers go brrr and no weird Seeker requests and responses needed (except with the coordinator ofc)

Key Files to Review

Group 1

  • File 1: seeker_coordinator.hpp and .cpp
  • File 2: seeker_client.cpp and .hpp

Group 2

  • File 3: offense.cpp and .hpp
  • File 4: Seeker coordinator .msg files

Review Checklist

  • Docstrings: All methods and classes should have the file appropriate docstrings which follow the guidelines in the "Contributing" page of our docs.
  • Remove extra print statements: Any print statements used for debugging should be removed
  • Tag reviewers: Tag some people for review and ping them on Slack

(Optional) Sub-issues (for drafts)

Note: if you find yourself breaking this PR into many smaller features, it may make sense to break up the PR into logical units based on these features.

  • Step 1
  • Step 2

Comment thread launch/soccer.launch.py Outdated
Comment thread src/rj_strategy/src/agent/position/offense.cpp Outdated
Comment thread src/rj_strategy/src/agent/position/offense.cpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeker_client.cpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeker_coordinator.cpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeking_coordinator.cpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeker_coordinator.cpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeker_coordinator.cpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeker_coordinator.cpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeking_coordinator.cpp
@jvogt23 jvogt23 requested a review from petergarud January 9, 2026 18:59

@Squid5678 Squid5678 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

some changes

Comment thread src/rj_strategy/include/rj_strategy/coordinator/seeking_client.hpp Outdated
Comment thread src/rj_strategy/include/rj_strategy/agent/position/offense.hpp
Comment thread src/rj_strategy/include/rj_strategy/coordinator/seeking_coordinator.hpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeking_client.cpp
Comment thread src/rj_strategy/src/coordinator/seeking_coordinator.cpp Outdated
Comment thread src/rj_strategy/src/coordinator/seeking_coordinator.cpp Outdated
@jvogt23 jvogt23 requested a review from Squid5678 February 8, 2026 20:45
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.

3 participants