openc3 5.12.0 → 5.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 056635373017dd511a8d049e778f59c7548c62dbae9b905effb03d447a0775ae
-  data.tar.gz: bfe11cbba1792a9741d06f0c3ea9365757d5a3fac0f873cb2a86597796521a11
+  metadata.gz: 70d4819352855e3fdb8b4c4d39331968e3e60af59beb0555743b42669d5b4b4a
+  data.tar.gz: c906a678c11f68b8751000875fe5cfdfdbbe98880e182edc572791246d625491
 SHA512:
-  metadata.gz: d113e832989cef2126d880d5bfb5281185bcf4d393066c8a58797c9178d49165be1c5179609baf665b4bc51d698c1dbd9ed038c580f20c66ac9bdd532eb03d15
-  data.tar.gz: 6c6010c3310998627247edb81bd3958206e1756d87973a0c601183bc3f50ad8fddd1a2d22fd85d2368c9094a59f7e27f1031b80606f4bce0839d373a71213cf5
+  metadata.gz: 2adcf16c91e84be3d1ef2b412ffc57eda1e7ed87e2d808519f9502c2cd937dcc06e766fad4f74b346daa0122a08745442decdbeb9185990206d7098c0331bf51
+  data.tar.gz: 413edafbcaebcb868c68f1e0ddfeadc4ef32e4950be652a2520858060efd2274d28e9f15f85d944ae48058c1c52cc72ad768f097f5a7830217ae71345869044d

data/Gemfile

@@ -14,4 +14,4 @@ gem 'tzinfo-data'
 gem 'rspec-rails', '~> 6.0'
 gem 'simplecov', '~> 0.20'
 gem 'simplecov-cobertura', '~> 2.1'
-gem 'mock_redis', '0.39'
+gem 'mock_redis', '0.40'

data/bin/openc3cli

@@ -42,9 +42,8 @@ require 'find'
 require 'json'
 require 'redis'
 require 'erb'
-require 'pp'
-require "irb"
-require "irb/completion"
+require 'irb'
+require 'irb/completion'
 
 $redis_url = "redis://#{ENV['OPENC3_REDIS_HOSTNAME']}:#{ENV['OPENC3_REDIS_PORT']}"
 
@@ -68,6 +67,7 @@ def print_usage
   puts "  cli validate /PATH/FILENAME.gem SCOPE variables.txt # Validate a COSMOS plugin gem file"
   puts "  cli load /PATH/FILENAME.gem SCOPE variables.txt     # Loads a COSMOS plugin gem file"
   puts "  cli generate TYPE OPTIONS         # Generate various COSMOS entities"
+  puts "    OPTIONS includes either --ruby or --python to specify the language in the generated code"
   puts "  #{MIGRATE_PARSER}"
   puts "  cli bridge CONFIG_FILENAME        # Run COSMOS host bridge"
   puts "  cli bridgegem gem_name variable1=value1 variable2=value2 # Runs bridge using gem bridge.txt"

data/data/config/graph_settings.yaml

@@ -18,7 +18,7 @@ HISTORY:
   parameters:
     - name: Value
       required: true
-      description: Value{d,h,m,s}. For example 1d, 2h, 30m, 15s
+      description: Value(d,h,m,s). For example 1d, 2h, 30m, 15s
       values: .+
 SECONDSGRAPHED:
   summary: Display the specified number of seconds in the graph

data/data/config/item_modifiers.yaml

@@ -32,8 +32,7 @@ STATE:
 READ_CONVERSION:
   summary: Applies a conversion to the current telemetry item
   description: Conversions are implemented in a custom Ruby file which should be
-    located in the target's lib folder and required by the target's target.txt
-    file (see REQUIRE). The class must require 'openc3/conversions/conversion'
+    located in the target's lib folder. The class must require 'openc3/conversions/conversion'
     and inherit from Conversion. It must implement the initialize method if it
     takes extra parameters and must always implement the call method. The conversion
     factor is applied to the raw value in the telemetry packet before it is displayed

data/data/config/parameter_modifiers.yaml

@@ -63,23 +63,22 @@ STATE:
       values: "['\"].*['\"]"
 WRITE_CONVERSION:
   summary: Applies a conversion when writing the current command parameter
-  description: Conversions are implemented in a custom Ruby file which should be
-    located in the target's lib folder and required by the target's target.txt
-    file (see REQUIRE). The class must require 'openc3/conversions/conversion'
+  description: |
+    Conversions are implemented in a custom Ruby file which should be
+    located in the target's lib folder. The class must require 'openc3/conversions/conversion'
     and inherit from Conversion. It must implement the initialize method if it
     takes extra parameters and must always implement the call method. The conversion
     factor is applied to the value entered by the user before it is written into
     the binary command packet and sent.
 
-    <div class="note info">
-    <h5>Multiple write conversions on command parameters</h5>
-    <p>When a command is built, each item gets written (and write conversions are run)
+    :::info Multiple write conversions on command parameters
+    When a command is built, each item gets written (and write conversions are run)
     to set the default value. Then items are written (again write conversions are run)
     with user provided values. Thus write conversions can be run twice. Also there are
     no guarantees which parameters have already been written. The packet itself has a
     given_values() method which can be used to retrieve a hash of the user provided
-    values to the command. That can be used to check parameter values passed in.</p>
-    </div>
+    values to the command. That can be used to check parameter values passed in.
+    :::
   example: |
     WRITE_CONVERSION the_great_conversion.rb 1000
 
@@ -154,7 +153,8 @@ SEG_POLY_WRITE_CONVERSION:
       values: .*
 GENERIC_WRITE_CONVERSION_START:
   summary: Start a generic write conversion
-  description: Adds a generic conversion function to the current command parameter.
+  description: |
+    Adds a generic conversion function to the current command parameter.
     This conversion factor is applied to the value entered by the user before it
     is written into the binary command packet and sent. The conversion is specified
     as ruby code that receives two implied parameters. 'value' which is the raw
@@ -164,15 +164,14 @@ GENERIC_WRITE_CONVERSION_START:
     value. The GENERIC_WRITE_CONVERSION_END keyword specifies that all lines of
     ruby code for the conversion have been given.
 
-    <div class="note info">
-    <h5>Multiple write conversions on command parameters</h5>
-    <p>When a command is built, each item gets written (and write conversions are run)
+    :::info Multiple write conversions on command parameters
+    When a command is built, each item gets written (and write conversions are run)
     to set the default value. Then items are written (again write conversions are run)
     with user provided values. Thus write conversions can be run twice. Also there are
     no guarantees which parameters have already been written. The packet itself has a
     given_values() method which can be used to retrieve a hash of the user provided
-    values to the command. That can be used to check parameter values passed in.</p>
-    </div>
+    values to the command. That can be used to check parameter values passed in.
+    :::
   warning: Generic conversions are not a good long term solution. Consider creating
     a conversion class and using WRITE_CONVERSION instead. WRITE_CONVERSION is easier
     to debug and higher performance.

data/data/config/screen.yaml

@@ -53,7 +53,7 @@ GLOBAL_SETTING:
       description: See SETTING for details.
       values: .*
   example: |
-    GLOBAL_SETTING LABELVALUELIMITSBAR COLORBLIND TRUE
+    GLOBAL_SETTING LABELVALUELIMITSBAR TEXTCOLOR BLACK
 GLOBAL_SUBSETTING:
   summary: Applies a widget subsetting to all widgets of a certain type
   description: Subsettings are only valid for widgets that are
@@ -81,7 +81,6 @@ GLOBAL_SUBSETTING:
       description: See SETTING for details.
       values: .*
   example: |
-    GLOBAL_SUBSETTING LABELVALUELIMITSBAR 1 COLORBLIND TRUE
     # Set all text color to white for labelvaluelimitsbars
     GLOBAL_SUBSETTING LABELVALUELIMITSBAR 0:0 TEXTCOLOR white
 SETTING:

data/data/config/target_config.yaml

@@ -13,10 +13,7 @@ LANGUAGE:
   since: 5.11.1
 REQUIRE:
   summary: Requires a Ruby file
-  description:
-    Ruby files must be required to be available to call in other code.
-    Files are first required from the target's lib folder. If no file is found the
-    Ruby system path is checked which includes the base openc3/lib folder.
+  description: Ruby files optionally be required to explicitly declare dependencies.
   example: REQUIRE limits_response.rb
   parameters:
     - name: Filename
@@ -25,8 +22,7 @@ REQUIRE:
         simply supply the filename, e.g. "REQUIRE my_file". Files in the base OpenC3
         lib directory also should just list the filename. If a file is in a folder
         under the lib directory then you must specify the folder name, e.g. "REQUIRE
-        folder/my_file". The filename can also be an absolute path but this is not
-        common. Note the ".rb" extension is optional when specifying the filename.
+        folder/my_file". Note the ".rb" extension is optional when specifying the filename.
       values: .+
 IGNORE_PARAMETER:
   summary: Ignore the given command parameter

data/lib/openc3/api/cmd_api.rb

@@ -39,12 +39,17 @@ module OpenC3
                        'cmd_raw_no_range_check',
                        'cmd_raw_no_hazardous_check',
                        'cmd_raw_no_checks',
-                       'build_command',
+                       'build_cmd',
+                       'build_command', # DEPRECATED
                        'send_raw',
-                       'get_all_commands',
-                       'get_all_command_names',
-                       'get_command',
-                       'get_parameter',
+                       'get_all_cmds',
+                       'get_all_commands', # DEPRECATED
+                       'get_all_cmd_names',
+                       'get_all_command_names', # DEPRECATED
+                       'get_cmd',
+                       'get_command', # DEPRECATED
+                       'get_param',
+                       'get_parameter', # DEPRECATED
                        'get_cmd_buffer',
                        'get_cmd_hazardous',
                        'get_cmd_value',
@@ -100,7 +105,7 @@ module OpenC3
     # Build a command binary
     #
     # @since 5.8.0
-    def build_command(*args, range_check: true, raw: false, scope: $openc3_scope, token: $openc3_token, **kwargs)
+    def build_cmd(*args, range_check: true, raw: false, scope: $openc3_scope, token: $openc3_token, **kwargs)
       extract_string_kwargs_to_args(args, kwargs)
       case args.length
       when 1
@@ -123,6 +128,8 @@ module OpenC3
       authorize(permission: 'cmd_info', target_name: target_name, scope: scope, token: token)
       DecomInterfaceTopic.build_cmd(target_name, cmd_name, cmd_params, range_check, raw, scope: scope)
     end
+    # build_command is DEPRECATED
+    alias build_command build_cmd
 
     # Send a raw binary string to the specified interface.
     #
@@ -140,9 +147,8 @@ module OpenC3
     # @param target_name [String] Target name of the command
     # @param command_name [String] Packet name of the command
     # @return [Hash] command hash with last command buffer
-    def get_cmd_buffer(target_name, command_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      command_name = command_name.upcase
+    def get_cmd_buffer(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, command_name = _extract_target_command_names('get_cmd_buffer', *args)
       authorize(permission: 'cmd_info', target_name: target_name, packet_name: command_name, scope: scope, token: token)
       TargetModel.packet(target_name, command_name, type: :CMD, scope: scope)
       topic = "#{scope}__COMMAND__{#{target_name}}__#{command_name}"
@@ -159,35 +165,38 @@ module OpenC3
     # @since 5.0.0
     # @param target_name [String] Name of the target
     # @return [Array<Hash>] Array of all commands as a hash
-    def get_all_commands(target_name, scope: $openc3_scope, token: $openc3_token)
+    def get_all_cmds(target_name, scope: $openc3_scope, token: $openc3_token)
       target_name = target_name.upcase
       authorize(permission: 'cmd_info', target_name: target_name, scope: scope, token: token)
       TargetModel.packets(target_name, type: :CMD, scope: scope)
     end
+    # get_all_commands is DEPRECATED
+    alias get_all_commands get_all_cmds
 
     # Returns an array of all the command packet names
     #
     # @since 5.0.6
     # @param target_name [String] Name of the target
     # @return [Array<String>] Array of all command packet names
-    def get_all_command_names(target_name, scope: $openc3_scope, token: $openc3_token)
+    def get_all_cmd_names(target_name, scope: $openc3_scope, token: $openc3_token)
       target_name = target_name.upcase
       authorize(permission: 'cmd_info', target_name: target_name, scope: scope, token: token)
       TargetModel.packet_names(target_name, type: :CMD, scope: scope)
     end
+    # get_all_command_names is DEPRECATED
+    alias get_all_command_names get_all_cmd_names
 
     # Returns a hash of the given command
     #
     # @since 5.0.0
-    # @param target_name [String] Name of the target
-    # @param command_name [String] Name of the packet
     # @return [Hash] Command as a hash
-    def get_command(target_name, command_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      command_name = command_name.upcase
+    def get_cmd(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, command_name = _extract_target_command_names('get_cmd', *args)
       authorize(permission: 'cmd_info', target_name: target_name, scope: scope, token: token)
       TargetModel.packet(target_name, command_name, type: :CMD, scope: scope)
     end
+    # get_command is DEPRECATED
+    alias get_command get_cmd
 
     # Returns a hash of the given command parameter
     #
@@ -196,13 +205,13 @@ module OpenC3
     # @param command_name [String] Name of the packet
     # @param parameter_name [String] Name of the parameter
     # @return [Hash] Command parameter as a hash
-    def get_parameter(target_name, command_name, parameter_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      command_name = command_name.upcase
-      parameter_name = parameter_name.upcase
+    def get_param(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, command_name, parameter_name = _extract_target_command_parameter_names('get_param', *args)
       authorize(permission: 'cmd_info', target_name: target_name, packet_name: command_name, scope: scope, token: token)
       TargetModel.packet_item(target_name, command_name, parameter_name, type: :CMD, scope: scope)
     end
+    # get_parameter is DEPRECATED
+    alias get_parameter get_param
 
     # Returns whether the specified command is hazardous
     #
@@ -257,19 +266,35 @@ module OpenC3
     end
 
     # Returns a value from the specified command
-    #
-    # @param target_name [String] Target name of the command
-    # @param command_name [String] Packet name of the command
-    # @param parameter_name [String] Parameter name in the command
-    # @param value_type [Symbol] How the values should be converted. Must be
-    #   one of {Packet::VALUE_TYPES}
-    # @return [Varies] value
-    def get_cmd_value(target_name, command_name, parameter_name, value_type = :CONVERTED, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      command_name = command_name.upcase
-      parameter_name = parameter_name.upcase
+    # Supports the following call syntax:
+    #   get_cmd_value("TGT PKT ITEM", type: :RAW)
+    #   get_cmd_value("TGT", "PKT", "ITEM", type: :RAW)
+    #   get_cmd_value("TGT", "PKT", "ITEM", :RAW) # DEPRECATED
+    def get_cmd_value(*args, type: :CONVERTED, scope: $openc3_scope, token: $openc3_token)
+      target_name = nil
+      command_name = nil
+      parameter_name = nil
+      case args.length
+      when 1
+        target_name, command_name, parameter_name = args[0].upcase.split
+      when 3
+        target_name = args[0].upcase
+        command_name = args[1].upcase
+        parameter_name = args[2].upcase
+      when 4
+        target_name = args[0].upcase
+        command_name = args[1].upcase
+        parameter_name = args[2].upcase
+        type = args[3].upcase
+      else
+        # Invalid number of arguments
+        raise "ERROR: Invalid number of arguments (#{args.length}) passed to get_cmd_value()"
+      end
+      if target_name.nil? or command_name.nil? or parameter_name.nil?
+        raise "ERROR: Target name, command name and parameter name required. Usage: get_cmd_value(\"TGT CMD PARAM\") or #{method_name}(\"TGT\", \"CMD\", \"PARAM\")"
+      end
       authorize(permission: 'cmd_info', target_name: target_name, packet_name: command_name, scope: scope, token: token)
-      CommandDecomTopic.get_cmd_item(target_name, command_name, parameter_name, type: value_type, scope: scope)
+      CommandDecomTopic.get_cmd_item(target_name, command_name, parameter_name, type: type, scope: scope)
     end
 
     # Returns the time the most recent command was sent
@@ -317,9 +342,8 @@ module OpenC3
     # @param target_name [String] Target name of the command
     # @param command_name [String] Packet name of the command
     # @return [Numeric] Transmit count for the command
-    def get_cmd_cnt(target_name, command_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      command_name = command_name.upcase
+    def get_cmd_cnt(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, command_name = _extract_target_command_names('get_cmd_cnt', *args)
       authorize(permission: 'system', target_name: target_name, packet_name: command_name, scope: scope, token: token)
       TargetModel.packet(target_name, command_name, type: :CMD, scope: scope)
       Topic.get_cnt("#{scope}__COMMAND__{#{target_name}}__#{command_name}")
@@ -347,6 +371,46 @@ module OpenC3
     # PRIVATE implementation details
     ###########################################################################
 
+    def _extract_target_command_names(method_name, *args)
+      target_name = nil
+      command_name = nil
+      case args.length
+      when 1
+        target_name, command_name = args[0].upcase.split
+      when 2
+        target_name = args[0].upcase
+        command_name = args[1].upcase
+      else
+        # Invalid number of arguments
+        raise "ERROR: Invalid number of arguments (#{args.length}) passed to #{method_name}()"
+      end
+      if target_name.nil? or command_name.nil?
+        raise "ERROR: Target name and command name required. Usage: #{method_name}(\"TGT CMD\") or #{method_name}(\"TGT\", \"CMD\")"
+      end
+      return [target_name, command_name]
+    end
+
+    def _extract_target_command_parameter_names(method_name, *args)
+      target_name = nil
+      command_name = nil
+      parameter_name = nil
+      case args.length
+      when 1
+        target_name, command_name, parameter_name = args[0].upcase.split
+      when 3
+        target_name = args[0].upcase
+        command_name = args[1].upcase
+        parameter_name = args[2].upcase
+      else
+        # Invalid number of arguments
+        raise "ERROR: Invalid number of arguments (#{args.length}) passed to #{method_name}()"
+      end
+      if target_name.nil? or command_name.nil? or parameter_name.nil?
+        raise "ERROR: Target name, command name and parameter name required. Usage: #{method_name}(\"TGT CMD PARAM\") or #{method_name}(\"TGT\", \"CMD\", \"PARAM\")"
+      end
+      return [target_name, command_name, parameter_name]
+    end
+
     def _cmd_implementation(method_name, *args, range_check:, hazardous_check:, raw:, timeout: nil, log_message: nil,
                            scope: $openc3_scope, token: $openc3_token, **kwargs)
       extract_string_kwargs_to_args(args, kwargs)

data/lib/openc3/api/tlm_api.rb

@@ -44,9 +44,12 @@ module OpenC3
                        'get_tlm_buffer',
                        'get_tlm_packet',
                        'get_tlm_values',
-                       'get_all_telemetry',
-                       'get_all_telemetry_names',
-                       'get_telemetry',
+                       'get_all_tlm',
+                       'get_all_telemetry', # DEPRECATED
+                       'get_all_tlm_names',
+                       'get_all_telemetry_names', # DEPRECATED
+                       'get_tlm',
+                       'get_telemetry', # DEPRECATED
                        'get_item',
                        'subscribe_packets',
                        'get_packets',
@@ -201,9 +204,8 @@ module OpenC3
     # @param target_name [String] Name of the target
     # @param packet_name [String] Name of the packet
     # @return [Hash] telemetry hash with last telemetry buffer
-    def get_tlm_buffer(target_name, packet_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      packet_name = packet_name.upcase
+    def get_tlm_buffer(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, packet_name = _extract_target_packet_names('get_tlm_buffer', *args)
       authorize(permission: 'tlm', target_name: target_name, packet_name: packet_name, scope: scope, token: token)
       TargetModel.packet(target_name, packet_name, scope: scope)
       topic = "#{scope}__TELEMETRY__{#{target_name}}__#{packet_name}"
@@ -224,9 +226,8 @@ module OpenC3
     # @return [Array<String, Object, Symbol|nil>] Returns an Array consisting
     #   of [item name, item value, item limits state] where the item limits
     #   state can be one of {OpenC3::Limits::LIMITS_STATES}
-    def get_tlm_packet(target_name, packet_name, stale_time: 30, type: :CONVERTED, cache_timeout: 0.1, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      packet_name = packet_name.upcase
+    def get_tlm_packet(*args, stale_time: 30, type: :CONVERTED, cache_timeout: 0.1, scope: $openc3_scope, token: $openc3_token)
+      target_name, packet_name = _extract_target_packet_names('get_tlm_packet', *args)
       authorize(permission: 'tlm', target_name: target_name, packet_name: packet_name, scope: scope, token: token)
       packet = TargetModel.packet(target_name, packet_name, scope: scope)
       t = _validate_tlm_type(type)
@@ -274,22 +275,24 @@ module OpenC3
     # @since 5.0.0
     # @param target_name [String] Name of the target
     # @return [Array<Hash>] Array of all telemetry packet hashes
-    def get_all_telemetry(target_name, scope: $openc3_scope, token: $openc3_token)
+    def get_all_tlm(target_name, scope: $openc3_scope, token: $openc3_token)
       target_name = target_name.upcase
       authorize(permission: 'tlm', target_name: target_name, scope: scope, token: token)
       TargetModel.packets(target_name, type: :TLM, scope: scope)
     end
+    alias get_all_telemetry get_all_tlm
 
     # Returns an array of all the telemetry packet names
     #
     # @since 5.0.6
     # @param target_name [String] Name of the target
     # @return [Array<String>] Array of all telemetry packet names
-    def get_all_telemetry_names(target_name, scope: $openc3_scope, token: $openc3_token)
+    def get_all_tlm_names(target_name, scope: $openc3_scope, token: $openc3_token)
       target_name = target_name.upcase
       authorize(permission: 'cmd_info', target_name: target_name, scope: scope, token: token)
       TargetModel.packet_names(target_name, type: :TLM, scope: scope)
     end
+    alias get_all_telemetry_names get_all_tlm_names
 
     # Returns a telemetry packet hash
     #
@@ -297,12 +300,12 @@ module OpenC3
     # @param target_name [String] Name of the target
     # @param packet_name [String] Name of the packet
     # @return [Hash] Telemetry packet hash
-    def get_telemetry(target_name, packet_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      packet_name = packet_name.upcase
+    def get_tlm(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, packet_name = _extract_target_packet_names('get_tlm', *args)
       authorize(permission: 'tlm', target_name: target_name, packet_name: packet_name, scope: scope, token: token)
       TargetModel.packet(target_name, packet_name, scope: scope)
     end
+    alias get_telemetry get_tlm
 
     # Returns a telemetry packet item hash
     #
@@ -311,10 +314,8 @@ module OpenC3
     # @param packet_name [String] Name of the packet
     # @param item_name [String] Name of the packet
     # @return [Hash] Telemetry packet item hash
-    def get_item(target_name, packet_name, item_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      packet_name = packet_name.upcase
-      item_name = item_name.upcase
+    def get_item(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, packet_name, item_name = _extract_target_packet_item_names('get_item', *args)
       authorize(permission: 'tlm', target_name: target_name, packet_name: packet_name, scope: scope, token: token)
       TargetModel.packet_item(target_name, packet_name, item_name, scope: scope)
     end
@@ -375,9 +376,8 @@ module OpenC3
     # @param target_name [String] Name of the target
     # @param packet_name [String] Name of the packet
     # @return [Numeric] Receive count for the telemetry packet
-    def get_tlm_cnt(target_name, packet_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      packet_name = packet_name.upcase
+    def get_tlm_cnt(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, packet_name = _extract_target_packet_names('get_tlm_cnt', *args)
       authorize(permission: 'system', target_name: target_name, packet_name: packet_name, scope: scope, token: token)
       TargetModel.packet(target_name, packet_name, scope: scope)
       Topic.get_cnt("#{scope}__TELEMETRY__{#{target_name}}__#{packet_name}")
@@ -403,9 +403,8 @@ module OpenC3
     # @param target_name [String] Target name
     # @param packet_name [String] Packet name
     # @return [Array<String>] All of the ignored telemetry items for a packet.
-    def get_packet_derived_items(target_name, packet_name, scope: $openc3_scope, token: $openc3_token)
-      target_name = target_name.upcase
-      packet_name = packet_name.upcase
+    def get_packet_derived_items(*args, scope: $openc3_scope, token: $openc3_token)
+      target_name, packet_name = _extract_target_packet_names('get_packet_derived_items', *args)
       authorize(permission: 'tlm', target_name: target_name, packet_name: packet_name, scope: scope, token: token)
       packet = TargetModel.packet(target_name, packet_name, scope: scope)
       return packet['items'].select { |item| item['data_type'] == 'DERIVED' }.map { |item| item['name'] }
@@ -413,6 +412,46 @@ module OpenC3
 
     # PRIVATE
 
+    def _extract_target_packet_names(method_name, *args)
+      target_name = nil
+      packet_name = nil
+      case args.length
+      when 1
+        target_name, packet_name = args[0].upcase.split
+      when 2
+        target_name = args[0].upcase
+        packet_name = args[1].upcase
+      else
+        # Invalid number of arguments
+        raise "ERROR: Invalid number of arguments (#{args.length}) passed to #{method_name}()"
+      end
+      if target_name.nil? or packet_name.nil?
+        raise "ERROR: Both target name and packet name required. Usage: #{method_name}(\"TGT PKT\") or #{method_name}(\"TGT\", \"PKT\")"
+      end
+      return [target_name, packet_name]
+    end
+
+    def _extract_target_packet_item_names(method_name, *args)
+      target_name = nil
+      packet_name = nil
+      item_name = nil
+      case args.length
+      when 1
+        target_name, packet_name, item_name = args[0].upcase.split
+      when 3
+        target_name = args[0].upcase
+        packet_name = args[1].upcase
+        item_name = args[2].upcase
+      else
+        # Invalid number of arguments
+        raise "ERROR: Invalid number of arguments (#{args.length}) passed to #{method_name}()"
+      end
+      if target_name.nil? or packet_name.nil? or item_name.nil?
+        raise "ERROR: Target name, packet name, and item name are required. Usage: #{method_name}(\"TGT PKT ITEM\") or #{method_name}(\"TGT\", \"PKT\", \"ITEM\")"
+      end
+      return [target_name, packet_name, item_name]
+    end
+
     def _validate_tlm_type(type)
       case type.intern
       when :RAW

data/lib/openc3/interfaces/mqtt_interface.rb

@@ -161,8 +161,8 @@ module OpenC3
     end
 
     def read
-      topic = @read_topics.shift
       packet = super()
+      topic = @read_topics.shift
       return nil unless packet
       identified_packet = @read_packets_by_topic[topic]
       if identified_packet
@@ -175,15 +175,17 @@ module OpenC3
     end
 
     def write(packet)
-      topics = packet.meta['TOPIC']
-      topics = packet.meta['TOPICS'] unless topics
-      if topics
-        topics.each do |topic|
-          @write_topics << topic
-          super(packet)
+      @write_mutex.synchronize do
+        topics = packet.meta['TOPIC']
+        topics = packet.meta['TOPICS'] unless topics
+        if topics
+          topics.each do |topic|
+            @write_topics << topic
+            super(packet)
+          end
+        else
+          raise "Command packet #{packet.target_name} #{packet.packet_name} requires a META TOPIC or TOPICS"
         end
-      else
-        raise "Command packet #{packet.target_name} #{packet.packet_name} requires a META TOPIC or TOPICS"
       end
     end