Skip to content

Fix online/offline state mechanism documentation#90

Merged
b-Tomas merged 1 commit intomainfrom
fix/online-offline-docs
Feb 26, 2026
Merged

Fix online/offline state mechanism documentation#90
b-Tomas merged 1 commit intomainfrom
fix/online-offline-docs

Conversation

@b-Tomas
Copy link
Member

@b-Tomas b-Tomas commented Feb 26, 2026

Description

Correct docstrings and comments to accurately describe how the get_state request mechanism works: InOrbit sends it when the robot is marked as offline but system stats are being received.

  • Clarify _handle_get_state trigger conditions
  • Remove misleading TODO about proactive status setting
  • Remove inaccurate fallback messages in _send_robot_status
  • Add system stats / get_state relationship to publish_system_stats docstring

Made with Cursor

Correct docstrings and comments to accurately describe how the
get_state request mechanism works: InOrbit sends it when the robot
is marked as offline but system stats are being received.

- Clarify _handle_get_state trigger conditions
- Remove misleading TODO about proactive status setting
- Remove inaccurate fallback messages in _send_robot_status
- Add system stats / get_state relationship to publish_system_stats

Made-with: Cursor
@b-Tomas b-Tomas requested a review from miguelgarcia February 26, 2026 13:11
@b-Tomas b-Tomas self-assigned this Feb 26, 2026
@b-Tomas b-Tomas added the documentation Improvements or additions to documentation label Feb 26, 2026
Copy link
Member

@miguelgarcia miguelgarcia left a comment

Choose a reason for hiding this comment

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

LGTM

@b-Tomas b-Tomas merged commit 549153c into main Feb 26, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants