fix: correct rclpy logger calls to use f-strings (pre-existing bugs)

rclpy RcutilsLogger.info/warning/debug() do not accept printf-style positional format args. Also fix p["use_phone_timestamp"] → p["use_phone_ts"] key mismatch in __init__ log line. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
feat: Add iOS phone GPS MQTT-to-ROS2 bridge topic (Issue #681 )
2026-04-04 11:20:58 -04:00 · 2026-04-04 11:20:58 -04:00 · 2026-04-04 11:15:13 -04:00 · 2026-04-04 11:15:12 -04:00 · 2026-04-04 10:58:09 -04:00 · 2026-04-04 10:56:06 -04:00
1 changed files with 7 additions and 1 deletions
--- a/jetson/ros2_ws/src/saltybot_bringup/config/rosbridge_params.yaml
+++ b/jetson/ros2_ws/src/saltybot_bringup/config/rosbridge_params.yaml
@ -54,6 +54,12 @@ rosbridge_websocket:
       "/vesc/right/state",
       "/tf",
       "/tf_static",
       "/saltybot/phone/gps",
       "/saltybot/phone/imu",
       "/saltybot/phone/battery",
       "/saltybot/phone/bridge/status",
       "/gps/fix",
       "/gps/vel",
       "/saltybot/ios/gps"]
    services_glob: "[]"       # no service calls via WebSocket
@ -68,7 +74,7 @@ rosbridge_websocket:
    # Delay between consecutive outgoing messages (ms). 0 = unlimited.
    # Set > 0 (e.g. 10) if browser JS event loop is overwhelmed.
-    delay_between_messages: 0
+    delay_between_messages: 0.0
    # ── Logging ───────────────────────────────────────────────────────────────
    # Set to true to log every publish/subscribe call (verbose, dev only).
Author	SHA1	Message	Date
sl-jetson	fbc88f5c2a	fix: correct rclpy logger calls to use f-strings (pre-existing bugs) rclpy RcutilsLogger.info/warning/debug() do not accept printf-style positional format args. Also fix p["use_phone_timestamp"] → p["use_phone_ts"] key mismatch in __init__ log line. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-04-04 11:20:58 -04:00
sl-jetson	0122957b6b	feat: Add iOS phone GPS MQTT-to-ROS2 bridge topic (Issue #681 ) - Add _TOPIC_IOS_GPS = 'saltybot/ios/gps' constant - Subscribe to saltybot/ios/gps in _on_mqtt_connect - Dispatch to _handle_ios_gps() in _dispatch() - _handle_ios_gps(): same logic as _handle_gps(), frame_id='ios_gps', publishes to /saltybot/ios/gps via self._ios_gps_pub - Add rx/pub/err/last_rx_ts counters for the new topic - Add /saltybot/ios/gps to rosbridge_params.yaml topics_glob Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-04-04 11:20:58 -04:00
sl-jetson	b1e8da4403	Merge pull request 'feat: iOS phone GPS via rosbridge topic /saltybot/ios/gps (Issue #681 )' (#722 ) from sl-webui/issue-681-ios-gps-rosbridge into main	2026-04-04 11:15:13 -04:00
sl-jetson	dd8afb480f	Merge pull request 'fix: add phone bridge and GPS topics to rosbridge whitelist (Issue #681 )' (#721 ) from sl-webui/issue-681-fix-gps-topics into main	2026-04-04 11:15:12 -04:00
sl-jetson	416a393134	fix: correct delay_between_messages type to float in rosbridge_params rclpy expects DOUBLE for this param; integer 0 raises InvalidParameterTypeException. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-04-04 10:58:09 -04:00
sl-jetson	60f500c206	fix: add phone bridge and GPS topics to rosbridge whitelist (Issue #681 ) Add /saltybot/phone/gps, /saltybot/phone/imu, /saltybot/phone/battery, /saltybot/phone/bridge/status, /gps/fix, /gps/vel to topics_glob so the browser GPS dashboard can receive phone-bridged GPS data. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-04-04 10:56:06 -04:00