Add --hide-unmonitored arg to frontend

Signed-off-by: Blake McHale <[email protected]>

Author: bmchalenv

greenwave_monitor/greenwave_monitor/ncurses_frontend.py

@@ -24,6 +24,7 @@
 with real-time diagnostics including publication rates, latency, and status.
 """
 
+import argparse
 import curses
 import signal
 import threading
@@ -40,7 +41,7 @@
 class GreenwaveNcursesFrontend(Node):
     """Ncurses frontend for Greenwave Monitor."""
 
-    def __init__(self):
+    def __init__(self, hide_unmonitored: bool = False):
         """Initialize the ncurses frontend node."""
         super().__init__('greenwave_ncurses_frontend')
 
@@ -59,7 +60,7 @@ def __init__(self):
         self.input_buffer = ''
         self.status_message = ''
         self.status_timeout = 0
-        self.show_only_monitored = False
+        self.show_only_monitored = hide_unmonitored
 
         # Initialize UI adaptor
         self.ui_adaptor = GreenwaveUiAdaptor(self)
@@ -448,10 +449,31 @@ def curses_main(stdscr, node):
         stdscr.refresh()
 
 
+def parse_args():
+    """Parse command-line arguments."""
+    parser = argparse.ArgumentParser(
+        description='Ncurses-based frontend for Greenwave Monitor',
+        add_help=False
+    )
+    parser.add_argument(
+        '--help',
+        action='help',
+        default=argparse.SUPPRESS,
+        help='Show this help message and exit'
+    )
+    parser.add_argument(
+        '--hide-unmonitored',
+        action='store_true',
+        help='Hide unmonitored topics on initialization'
+    )
+    return parser.parse_known_args()
+
+
 def main(args=None):
     """Entry point for the ncurses frontend application."""
-    rclpy.init(args=args)
-    node = GreenwaveNcursesFrontend()
+    parsed_args, ros_args = parse_args()
+    rclpy.init(args=ros_args)
+    node = GreenwaveNcursesFrontend(hide_unmonitored=parsed_args.hide_unmonitored)
     thread = None
 
     def signal_handler(signum, frame):

greenwave_monitor/scripts/ncurses_dashboard

@@ -23,6 +23,7 @@
 # Parse command line arguments
 DEMO_MODE=false
 LOG_DIR=""
+HIDE_UNMONITORED=false
 MONITOR_ARGS=()
 
 show_help() {
@@ -33,6 +34,7 @@ show_help() {
     echo "OPTIONS:"
     echo "  --demo, --test             Launch demo publisher nodes for testing"
     echo "  --log-dir DIR              Enable logging to specified directory"
+    echo "  --hide-unmonitored         Hide unmonitored topics on initialization"
     echo "  --help, -h                 Show this help message"
     echo ""
     echo "MONITOR_ARGS are passed directly to the greenwave_monitor node"
@@ -56,6 +58,10 @@ while [[ $# -gt 0 ]]; do
             LOG_DIR="$2"
             shift 2
             ;;
+        --hide-unmonitored)
+            HIDE_UNMONITORED=true
+            shift
+            ;;
         --help|-h)
             show_help
             exit 0
@@ -115,7 +121,11 @@ echo "Monitor process started with PID: $MONITOR_PID"
 # Launch ncurses frontend in the foreground
 echo "Starting ncurses TUI..."
 echo "Controls: a=Add Topic, r=Remove, f=Set Frequency, c=Clear Freq, q=Quit"
-python3 -m greenwave_monitor.ncurses_frontend
+FRONTEND_ARGS=()
+if [ "$HIDE_UNMONITORED" = "true" ]; then
+    FRONTEND_ARGS+=("--hide-unmonitored")
+fi
+python3 -m greenwave_monitor.ncurses_frontend "${FRONTEND_ARGS[@]}"
 
 # Note: We don't need to explicitly exit here because the trap will handle cleanup
 # when the ncurses frontend exits

greenwave_monitor/test/test_ncurses_frontend.py

@@ -0,0 +1,53 @@
+#!/usr/bin/env python3
+
+# SPDX-FileCopyrightText: NVIDIA CORPORATION & AFFILIATES
+# Copyright (c) 2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+# SPDX-License-Identifier: Apache-2.0
+
+"""Tests for ncurses frontend argument parsing."""
+
+import sys
+
+
+class TestParseArgs:
+    """Test argument parsing for ncurses frontend."""
+
+    def test_default_hide_unmonitored_false(self, monkeypatch):
+        """Test that hide_unmonitored defaults to False."""
+        monkeypatch.setattr(sys, 'argv', ['ncurses_frontend'])
+        from greenwave_monitor.ncurses_frontend import parse_args
+        parsed_args, _ = parse_args()
+        assert parsed_args.hide_unmonitored is False
+
+    def test_hide_unmonitored_long_flag(self, monkeypatch):
+        """Test --hide-unmonitored flag enables hide_unmonitored."""
+        monkeypatch.setattr(sys, 'argv', ['ncurses_frontend', '--hide-unmonitored'])
+        from greenwave_monitor.ncurses_frontend import parse_args
+        parsed_args, _ = parse_args()
+        assert parsed_args.hide_unmonitored is True
+
+    def test_ros_args_passthrough(self, monkeypatch):
+        """Test that ROS arguments are passed through."""
+        monkeypatch.setattr(
+            sys, 'argv',
+            ['ncurses_frontend', '--hide-unmonitored', '--ros-args', '-r', '__node:=my_node']
+        )
+        from greenwave_monitor.ncurses_frontend import parse_args
+        parsed_args, ros_args = parse_args()
+        assert parsed_args.hide_unmonitored is True
+        assert '--ros-args' in ros_args
+        assert '-r' in ros_args
+        assert '__node:=my_node' in ros_args