cosmos 4.4.0-java → 4.5.1-java

data/lib/cosmos/tools/cmd_sequence/sequence_item.rb

@@ -14,7 +14,7 @@ Cosmos.catch_fatal_exception do
   require 'cosmos/script'
   require 'cosmos/gui/dialogs/calendar_dialog'
   require 'cosmos/gui/dialogs/cmd_details_dialog'
-  require 'cosmos/tools/cmd_sender/cmd_param_table_item_delegate'
+  require 'cosmos/tools/cmd_sender/cmd_params'
 end
 
 module Cosmos
@@ -26,36 +26,18 @@ module Cosmos
     signals 'modified()'
     MANUALLY = "MANUALLY ENTERED"
 
-    # Parse a time and command string into a SequenceItem which is returned
-    # @param time [String] Time (absolute or relative delay). Supports a single
-    #   float value (relative delay) or a absolute time specified as
-    #   "YYYY/MM/DD HH:MM:SS.MS"
-    # @param command [String] Command String which should not be
-    #   quoted and is everything inside the quotes of a command.
-    #   For example: TGT PKT with STRING 'HI', VALUE 12. Parse errors are
-    #   raised as execptions which must be handled by higher level code.
-    # @return [SequenceItem] SequenceItem which the line represents
-    def self.parse(time, command)
-      tgt_name, pkt_name, cmd_params = extract_fields_from_cmd_text(command)
-      packet = System.commands.packet(tgt_name, pkt_name).dup
-      packet.restore_defaults
-      cmd_params.each do |param_name, param_value|
-        packet.write(param_name, param_value)
-      end
-      SequenceItem.new(packet, time)
-    end
-
     # Create a new SequenceItem based on the given command with the given delay
-    # @param command [Packet] Command packet
     # @param time [String] Absolute time in YYYY/MM/DD HH:MM:SS format or a
     #   single float value representing the delta delay time
-    def initialize(command, time = nil)
+    def initialize(parent, target_name, packet_name, params = nil, time = nil)
       super()
-      @command = command
-      @table = nil
-      @param_widgets = []
-      @show_ignored = false
-      @states_in_hex = false
+      @cmd_params = CmdParams.new
+      # Propagate the modified signal up
+      @cmd_params.connect(SIGNAL('modified()')) do
+        set_cmd_name_info()
+        emit modified
+      end
+      @command = System.commands.packet(target_name, packet_name)
       @expanded = false
       @file_dir = System.paths['LOGS']
 
@@ -66,50 +48,30 @@ module Cosmos
       top_layout = Qt::VBoxLayout.new
       top_layout.setContentsMargins(2, 0, 0, 0)
       setLayout(top_layout)
-      top_layout.addLayout(create_cmd_layout(command, time))
+      top_layout.addLayout(create_cmd_layout(target_name, packet_name, time))
       top_layout.addWidget(create_parameters())
-      update_cmd_params()
+      add_table(@cmd_params.update_cmd_params(@command, existing: params))
       set_cmd_name_info()
     end
 
-    # Set or clear read only status on the item
-    # @param bool [Boolean] Whether to make the item read only
-    def read_only(bool)
-      @time.setReadOnly(bool)
+    def add_table(table)
+      return unless table
+      @table_layout.addWidget(table) 
     end
-
-    # Show or hide ignored parameters
-    # @param bool [Boolean] Whether to show ignored command items
-    def show_ignored(bool)
-      @show_ignored = bool
-      update_cmd_params(bool)
+      
+    def states_in_hex(checked)
+      @cmd_params.states_in_hex(checked)
     end
 
-    # Display state values in hex (or decimal)
-    # @param bool [Boolean] Whether to display state values in hex
-    def states_in_hex(bool)
-      @states_in_hex = bool
-      @param_widgets.each do |_, _, state_value_item|
-        next unless state_value_item
-        text = state_value_item.text
-        quotes_removed = text.remove_quotes
-        if text == quotes_removed
-          if bool
-            if text.is_int?
-              @table.blockSignals(true)
-              state_value_item.text = sprintf("0x%X", text.to_i)
-              @table.blockSignals(false)
-            end
-          else
-            if text.is_hex?
-              @table.blockSignals(true)
-              state_value_item.text = Integer(text).to_s
-              @table.blockSignals(false)
-            end
-          end
-        end
-      end
+    def show_ignored(checked)
+      add_table(@cmd_params.update_cmd_params(@command, show_ignored: checked))
+      set_cmd_name_info()
+    end
 
+    # Set or clear read only status on the item
+    # @param bool [Boolean] Whether to make the item read only
+    def read_only(bool)
+      @time.setReadOnly(bool)
     end
 
     # Show the command parameters part of the GUI
@@ -124,33 +86,9 @@ module Cosmos
       @parameters.hide
     end
 
-    # @return [Hash] All the parameter item values keyed by their name
-    def command_params
-      params = {}
-      @param_widgets.each do |packet_item, value_item, state_value_item|
-        text = ''
-        Qt.execute_in_main_thread do
-          text = value_item.text
-          text = state_value_item.text if state_value_item && (text == MANUALLY)
-        end
-        quotes_removed = text.remove_quotes
-        if text == quotes_removed
-          if (packet_item.data_type == :STRING or packet_item.data_type == :BLOCK) and text.upcase.start_with?("0X")
-            params[packet_item.name] = text.hex_to_byte_string
-          else
-            params[packet_item.name] = text.convert_to_value
-          end
-        else
-          params[packet_item.name] = quotes_removed
-        end
-        raise "#{packet_item.name} is required" if quotes_removed == '' && packet_item.required
-      end
-      params
-    end
-
     # @return [String] Command to be executed with no quotes or other decorations
     def command_string
-      output_string =  System.commands.build_cmd_output_string(@command.target_name, @command.packet_name, command_params(), false)
+      output_string =  System.commands.build_cmd_output_string(@command.target_name, @command.packet_name, @cmd_params.params_text, false)
       if output_string =~ /[\x00-\x08\x0B-\x0C\x0E-\x1F\x7F-\xFF]/
         output_string = output_string.inspect.remove_quotes
       end
@@ -165,7 +103,7 @@ module Cosmos
     end
 
     def command
-      System.commands.build_cmd(@command.target_name, @command.packet_name, command_params(), false)
+      System.commands.build_cmd(@command.target_name, @command.packet_name, @cmd_params.params_text, false)
     end
 
     # @return [String] Time and command string
@@ -193,12 +131,12 @@ module Cosmos
     # and the delete button to remove the sequence item.
     # @param command [Packet] Command packet
     # @param time [String] Execution delay in absolute or relative time
-    def create_cmd_layout(command, time)
+    def create_cmd_layout(target_name, packet_name, time)
       cmd_layout = Qt::HBoxLayout.new
       cmd_layout.setContentsMargins(0, 0, 0, 0)
       cmd_layout.addWidget(create_time_edit(time))
 
-      @cmd_name = Qt::Label.new("#{command.target_name} #{command.packet_name}")
+      @cmd_name = Qt::Label.new("#{target_name} #{packet_name}")
       cmd_layout.addWidget(@cmd_name)
       @cmd_info = Qt::Label.new("") # Label for the hazardous designation
       cmd_layout.addWidget(@cmd_info)
@@ -263,252 +201,21 @@ module Cosmos
       @parameters
     end
 
-    # Update the command parameters table for the given command
-    # @param ignored_toggle [Boolean] Whether to display the ignored
-    #   parameters. Pass nil (the default) to keep the existing setting.
-    def update_cmd_params(ignored_toggle = nil)
-      old_params = {}
-      if !ignored_toggle.nil?
-        # Save parameter values
-        @param_widgets.each do |packet_item, value_item, state_value_item|
-          text = value_item.text
-          if state_value_item
-            old_params[packet_item.name] = [text, state_value_item.text]
-          else
-            old_params[packet_item.name] = text
-          end
-        end
-      end
-
-      target = System.targets[@command.target_name]
-      packet_items = @command.sorted_items
-      shown_packet_items = []
-      packet_items.each do |packet_item|
-        if target && target.ignored_parameters.include?(packet_item.name) && !@show_ignored
-          if @param_widgets.empty? # First time rendering the parameters
-            if packet_item.states
-              # Skip this if the default matches the saved value
-              next if @command.read_item(packet_item, :RAW) == packet_item.default
-            else
-              # Skip this if the default matches the saved value
-              next if @command.read_item(packet_item) == packet_item.default
-            end
-          else # Check the current values
-            result = @param_widgets.select {|item,_,_| item == packet_item }
-            next if result.empty?
-            _, value_item, state_value_item = result[0]
-            value = state_value_item ? state_value_item.text : value_item.text
-            # Skip this if the default matches the current value
-            next if packet_item.default.to_s == value
-          end
-        end
-        shown_packet_items << packet_item
-      end
-
-      @table.dispose if @table
-      @table = nil
-
-      # Update Parameters
-      @param_widgets = []
-      drawn_header = false
-
-      row = 0
-      shown_packet_items.each do |packet_item|
-        value_item = nil
-        state_value_item = nil
-
-        unless drawn_header
-          @table = Qt::TableWidget.new()
-          @table.setSizePolicy(Qt::SizePolicy::Expanding, Qt::SizePolicy::Expanding)
-          @table.setWordWrap(true)
-          @table.setRowCount(shown_packet_items.length)
-          @table.setColumnCount(5)
-          @table.setHorizontalHeaderLabels(['Name', '         Value or State         ', '         ', 'Units', 'Description'])
-          @table.verticalHeader.setVisible(false)
-          @table.setItemDelegate(CmdParamTableItemDelegate.new(@table, @param_widgets))
-          @table.setEditTriggers(Qt::AbstractItemView::DoubleClicked | Qt::AbstractItemView::SelectedClicked | Qt::AbstractItemView::AnyKeyPressed)
-          @table.setSelectionMode(Qt::AbstractItemView::NoSelection)
-          @table.setContextMenuPolicy(Qt::CustomContextMenu)
-          @table.connect(SIGNAL('customContextMenuRequested(const QPoint&)')) do |point|
-            context_menu(point)
-          end
-          drawn_header = true
-        end
-
-        # Parameter Name
-        item = Qt::TableWidgetItem.new("#{packet_item.name}:")
-        item.setTextAlignment(Qt::AlignRight | Qt::AlignVCenter)
-        item.setFlags(Qt::NoItemFlags | Qt::ItemIsSelectable | Qt::ItemIsEnabled)
-        @table.setItem(row, 0, item)
-
-        if packet_item.states
-          default = @command.read_item(packet_item, :RAW)
-          default_state = packet_item.states.key(default)
-          if old_params[packet_item.name]
-            value_item = Qt::TableWidgetItem.new(old_params[packet_item.name][0])
-          else
-            if default_state
-              value_item = Qt::TableWidgetItem.new(default_state.to_s)
-            else
-              value_item = Qt::TableWidgetItem.new(MANUALLY)
-            end
-          end
-          value_item.setTextAlignment(Qt::AlignRight | Qt::AlignVCenter)
-          value_item.setFlags(Qt::NoItemFlags | Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsEditable)
-          @table.setItem(row, 1, value_item)
-
-          if old_params[packet_item.name]
-            state_value_item = Qt::TableWidgetItem.new(old_params[packet_item.name][1])
-          else
-            if @states_in_hex && packet_item.default.kind_of?(Integer)
-              state_value_item = Qt::TableWidgetItem.new(sprintf("0x%X", default))
-            else
-              default_str = default.to_s
-              if default_str.is_printable?
-                state_value_item = Qt::TableWidgetItem.new(default_str)
-              else
-                state_value_item = Qt::TableWidgetItem.new("0x" + default_str.simple_formatted)
-              end
-            end
-          end
-          state_value_item.setTextAlignment(Qt::AlignRight | Qt::AlignVCenter)
-          state_value_item.setFlags(Qt::NoItemFlags | Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsEditable)
-          @table.setItem(row, 2, state_value_item)
-        else # Parameter Value
-          if old_params[packet_item.name]
-            value_item = Qt::TableWidgetItem.new(old_params[packet_item.name])
-          else
-            default = @command.read_item(packet_item)
-            if packet_item.format_string
-              begin
-                value_text = sprintf(packet_item.format_string, default)
-              rescue
-                # Oh well - Don't use the format string
-                value_text = default.to_s
-              end
-            else
-              value_text = default.to_s
-            end
-            if !value_text.is_printable?
-              value_text = "0x" + value_text.simple_formatted
-            end
-            value_item = Qt::TableWidgetItem.new(value_text)
-          end
-          value_item.setTextAlignment(Qt::AlignRight | Qt::AlignVCenter)
-          value_item.setFlags(Qt::NoItemFlags | Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsEditable)
-          @table.setItem(row, 1, value_item)
-          @table.setSpan(row, 1, 1, 2)
-        end
-
-        # Units
-        item = Qt::TableWidgetItem.new(packet_item.units.to_s)
-        item.setTextAlignment(Qt::AlignRight | Qt::AlignVCenter)
-        item.setFlags(Qt::NoItemFlags | Qt::ItemIsSelectable | Qt::ItemIsEnabled)
-        @table.setItem(row, 3, item)
-
-        # Description
-        item = Qt::TableWidgetItem.new(packet_item.description.to_s)
-        item.setTextAlignment(Qt::AlignLeft | Qt::AlignVCenter)
-        item.setFlags(Qt::NoItemFlags | Qt::ItemIsSelectable | Qt::ItemIsEnabled)
-        @table.setItem(row, 4, item)
-
-        @param_widgets << [packet_item, value_item, state_value_item]
-        row += 1
-      end
-
-      if @table
-        connect_table_item_changed()
-        @table.setSizePolicy(Qt::SizePolicy.Minimum, Qt::SizePolicy.Minimum)
-        @table.setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOff)
-        @table.setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff)
-        @table.resizeColumnsToContents()
-        @table.resizeRowsToContents()
-        @table.setFixedSize(@table.horizontalHeader.length + @table.verticalHeader.width,
-                            2 + @table.verticalHeader.length + @table.horizontalHeader.height)
-        @table_layout.addWidget(@table)
-      end
-    end
-
     # Sets the @cmd_name label to the command that will be sent. Also udpates
     # the @cmd_info with whether this command is hazardous or not.
     def set_cmd_name_info
-      @cmd_name.text = command_string
-      hazardous, _ = System.commands.cmd_hazardous?(@command.target_name, @command.packet_name, command_params())
-      if hazardous
-        @cmd_info.text = "(Hazardous)"
-      else
-        @cmd_info.text = ""
+      Qt.execute_in_main_thread do
+        @cmd_name.text = command_string
+        hazardous, _ = System.commands.cmd_hazardous?(@command.target_name, @command.packet_name, @cmd_params.params_text)
+        if hazardous
+          @cmd_info.text = "(Hazardous)"
+        else
+          @cmd_info.text = ""
+        end
       end
     rescue => error
       @cmd_info.text = "(Error)"
-      Qt::MessageBox.warning(self, 'Error', error.message)
-    end
-
-    # If the user right clicks over a table item, this method displays a context
-    # menu with various options.
-    # @param point [Qt::Point] Point to display the context menu
-    def context_menu(point)
-      target_name = @command.target_name
-      packet_name = @command.packet_name
-      item = @table.itemAt(point)
-      if item
-        item_name = @table.item(item.row, 0).text[0..-2] # Remove :
-        if target_name.length > 0 && packet_name.length > 0 && item_name.length > 0
-          menu = Qt::Menu.new()
-
-          details_action = Qt::Action.new("Details #{target_name} #{packet_name} #{item_name}", self)
-          details_action.statusTip = "Popup details about #{target_name} #{packet_name} #{item_name}"
-          details_action.connect(SIGNAL('triggered()')) do
-            CmdDetailsDialog.new(nil, target_name, packet_name, item_name)
-          end
-          menu.addAction(details_action)
-
-          file_chooser_action = Qt::Action.new("Insert Filename", self)
-          file_chooser_action.statusTip = "Select a file and place its name into this parameter"
-          file_chooser_action.connect(SIGNAL('triggered()')) do
-              filename = Qt::FileDialog::getOpenFileName(self, "Insert Filename:", @file_dir, "All Files (*)")
-              if filename && !filename.empty?
-                @file_dir = File.dirname(filename)
-                _, value_item, state_value_item = @param_widgets[item.row]
-                if state_value_item
-                  state_value_item.setText(filename)
-                elsif value_item
-                  value_item.setText(filename)
-                end
-              end
-          end
-          menu.addAction(file_chooser_action)
-
-          menu.exec(@table.mapToGlobal(point))
-          menu.dispose
-        end
-      end # if item
-    end
-
-    # Connect the itemChanged signal to the table so we can handle user edits
-    # of the table parameters. This method emits the modified signal.
-    def connect_table_item_changed
-      @table.connect(SIGNAL('itemChanged(QTableWidgetItem*)')) do |item|
-        packet_item, value_item, state_value_item = @param_widgets[item.row]
-        if item.column == 1
-          if packet_item.states
-            value = packet_item.states[value_item.text]
-            @table.blockSignals(true)
-            if @states_in_hex && value.kind_of?(Integer)
-              state_value_item.setText(sprintf("0x%X", value))
-            else
-              state_value_item.setText(value.to_s)
-            end
-            @table.blockSignals(false)
-          end
-        elsif item.column == 2
-          @table.blockSignals(true)
-          @table.item(item.row, 1).setText(MANUALLY)
-          @table.blockSignals(false)
-        end
-        set_cmd_name_info()
-        emit modified # Tell the higher level that something changed
-      end
+      Qt::MessageBox.warning(self, 'Error', "Error parsing #{@command.target_name} #{@command.packet_name} due to #{error.message}")
     end
   end
 end

data/lib/cosmos/tools/cmd_sequence/sequence_list.rb

@@ -19,8 +19,9 @@ module Cosmos
     signals 'modified()'
 
     # Create the SequenceList
-    def initialize
+    def initialize(parent)
       super()
+      @parent = parent
       @modified = false
       layout = Qt::VBoxLayout.new()
       layout.setContentsMargins(0, 0, 0, 0)
@@ -45,7 +46,8 @@ module Cosmos
           usage = "#{keyword} <Delay Time> <Command>"
           parser.verify_num_parameters(2, 2, usage)
           begin
-            item = SequenceItem.parse(params[0], params[1])
+            tgt_name, pkt_name, cmd_params = extract_fields_from_cmd_text(params[1])
+            item = SequenceItem.new(@parent, tgt_name, pkt_name, cmd_params, params[0])
             # Connect the SequenceItems modified signal to propagate it
             # forward by emitting our own modified signal
             item.connect(SIGNAL("modified()")) do
@@ -63,12 +65,13 @@ module Cosmos
       @modified = false # Initially we're not modified
     end
 
-    # Add a new SequenceItem to the list.
-    # @param command [Packet] Command packet to base the SequenceItem on
+    # Add a new SequenceItem to the list based on the given target and packet
+    # @param target_name [String] target name containing the command
+    # @param packet_name [String] packet name containing the command
     # @return [SequenceItem] The item added
-    def add(command)
+    def add(target_name, packet_name)
       @modified = true
-      item = SequenceItem.new(command)
+      item = SequenceItem.new(@parent, target_name, packet_name)
       # Connect the SequenceItems modified signal to propagate it
       # forward by emitting our own modified signal
       item.connect(SIGNAL("modified()")) do
@@ -118,11 +121,13 @@ module Cosmos
     # application.
     # @param filename [String] Filename to open and write the sequence
     def save(filename)
-      @modified = false
-      File.open(filename, "w") do |file|
-        # Each SequenceItem's save method returns the save string
-        file.write(collect {|item| item.save }.join("\n"))
-        file.write("\n") # final newline
+      begin
+        sequence = collect {|item| item.save }.join("\n") + "\n"
+        @modified = false
+        File.open(filename, "w") {|file| file.write(sequence) }
+      rescue Exception => err
+        message = "Error saving due to #{err}"
+        Qt::MessageBox.critical(self, 'Error', message)
       end
     end
 

data/lib/cosmos/tools/cmd_tlm_server/api.rb

@@ -591,14 +591,16 @@ module Cosmos
       end
 
       # Update current value table
-      cvt_packet.buffer = packet.buffer(false)
-      cvt_packet.received_time = received_time
-
-      # The interface does the following line, but I don't think inject_tlm should because it could confuse the interface
-      target.tlm_cnt += 1
-      packet.received_count += 1
-      cvt_packet.received_count += 1
-      CmdTlmServer.instance.identified_packet_callback(packet)
+      cvt_packet.synchronize do
+        cvt_packet.buffer = packet.buffer(false)
+        cvt_packet.received_time = received_time
+
+        # The interface does the following line, but I don't think inject_tlm should because it could confuse the interface
+        target.tlm_cnt += 1
+        packet.received_count += 1
+        cvt_packet.received_count += 1
+        CmdTlmServer.instance.identified_packet_callback(cvt_packet)
+      end
 
       # Find the interface for this target
       interface = target.interface

data/lib/cosmos/tools/cmd_tlm_server/cmd_tlm_server.rb

@@ -201,9 +201,9 @@ module Cosmos
         @json_drb.method_whitelist = @api_whitelist
         begin
           if @mode == :CMD_TLM_SERVER
-            @json_drb.start_service(System.listen_hosts['CTS_API'], System.ports['CTS_API'], self)
+            @json_drb.start_service(System.listen_hosts['CTS_API'], System.ports['CTS_API'], self, 1000, System)
           else
-            @json_drb.start_service(System.listen_hosts['REPLAY_API'], System.ports['REPLAY_API'], self)
+            @json_drb.start_service(System.listen_hosts['REPLAY_API'], System.ports['REPLAY_API'], self, 1000, System)
           end
         rescue Exception
           # Call packet_logging shutdown here to explicitly kill the logging

data/lib/cosmos/tools/cmd_tlm_server/cmd_tlm_server_gui.rb

@@ -191,6 +191,7 @@ module Cosmos
 
       # File actions
       @file_reload = Qt::Action.new('&Reload Configuration', self)
+      @file_reload.shortcut = Qt::KeySequence.new('Ctrl+Shift+R')
       @file_reload.statusTip = 'Reload configuraton and reset'
       @file_reload.connect(SIGNAL('triggered()')) do
         CmdTlmServer.instance.reload()

data/lib/cosmos/tools/cmd_tlm_server/gui/logging_tab.rb

@@ -96,7 +96,7 @@ module Cosmos
         button_layout.addWidget(button, location[0], location[1])
         button.connect(SIGNAL('clicked()')) do
           begin
-            CmdTlmServer.instance.send(method, 'ALL')
+            CmdTlmServer.instance.public_send(method, 'ALL')
           rescue Exception => error
             statusBar.showMessage(error.message)
           end

data/lib/cosmos/tools/cmd_tlm_server/interface_thread.rb

@@ -148,51 +148,54 @@ module Cosmos
         if packet.identified?
           begin
             # Preidentifed packet - place it into the current value table
-            identified_packet = System.telemetry.update!(packet.target_name,
-                                                         packet.packet_name,
-                                                         packet.buffer)
+            identified_packet = System.telemetry.packet(packet.target_name, packet.packet_name)
           rescue RuntimeError
             # Packet identified but we don't know about it
             # Clear packet_name and target_name and try to identify
             Logger.warn "Received unknown identified telemetry: #{packet.target_name} #{packet.packet_name}"
             packet.target_name = nil
             packet.packet_name = nil
-            identified_packet = System.telemetry.identify!(packet.buffer,
-                                                           @interface.target_names)
+            identified_packet = System.telemetry.identify(packet.buffer, @interface.target_names)
           end
         else
           # Packet needs to be identified
-          identified_packet = System.telemetry.identify!(packet.buffer,
-                                                         @interface.target_names)
+          identified_packet = System.telemetry.identify(packet.buffer, @interface.target_names)
         end
       end
 
-      if identified_packet
+      unknown = false
+      unless identified_packet
+        unknown = true
+        identified_packet = System.telemetry.packet('UNKNOWN', 'UNKNOWN')
+      end
+      
+      identified_packet.synchronize do
+        identified_packet.buffer = packet.buffer unless packet.stored
         identified_packet.received_time = packet.received_time
         identified_packet.stored = packet.stored
         identified_packet.extra = packet.extra
         packet = identified_packet
-      else
-        unknown_packet = System.telemetry.update!('UNKNOWN', 'UNKNOWN', packet.buffer)
-        unknown_packet.received_time = packet.received_time
-        unknown_packet.stored = packet.stored
-        unknown_packet.extra = packet.extra
-        packet = unknown_packet
-        data_length = packet.length
-        string = "#{@interface.name} - Unknown #{data_length} byte packet starting: "
-        num_bytes_to_print = [UNKNOWN_BYTES_TO_PRINT, data_length].min
-        data_to_print = packet.buffer(false)[0..(num_bytes_to_print - 1)]
-        data_to_print.each_byte do |byte|
-          string << sprintf("%02X", byte)
+        
+        if unknown
+          data_length = packet.length
+          string = "#{@interface.name} - Unknown #{data_length} byte packet starting: "
+          num_bytes_to_print = [UNKNOWN_BYTES_TO_PRINT, data_length].min
+          data_to_print = packet.buffer(false)[0..(num_bytes_to_print - 1)]
+          data_to_print.each_byte do |byte|
+            string << sprintf("%02X", byte)
+          end
+          Logger.error string
         end
-        Logger.error string
-      end
 
-      target = System.targets[packet.target_name]
-      target.tlm_cnt += 1 if target
-      packet.received_count += 1
-      @identified_packet_callback.call(packet) if @identified_packet_callback
+        target = System.targets[packet.target_name]
+        target.tlm_cnt += 1 if target
+        packet.received_count += 1
+        @identified_packet_callback.call(packet) if @identified_packet_callback
 
+        # So we can release the mutex
+        packet = packet.clone
+      end
+      
       # Write to routers
       @interface.routers.each do |router|
         begin

data/lib/cosmos/tools/cmd_tlm_server/limits_groups_background_task.rb

@@ -86,7 +86,7 @@ module Cosmos
       return if @sleeper.sleep(@initial_delay)
       loop do
         start = Time.now
-        check_methods.each {|method| self.send(method.intern) }
+        check_methods.each {|method| self.public_send(method.intern) }
         now = Time.now
         @status = "#{now.formatted}: Checking groups took #{now - start}s"
         sleep_time = @task_delay - (now - start)

data/lib/cosmos/tools/cmd_tlm_server/router_thread.rb

@@ -19,6 +19,11 @@ module Cosmos
     protected
 
     def handle_packet(packet)
+      unless System.allow_router_commanding
+        Logger.error "Router received command with router commanding disabled"
+        return
+      end
+
       # Start out assuming we will route to all associated interfaces
       interfaces = @interface.interfaces