cosmos 3.8.2 → 3.8.3

data/lib/cosmos/packets/packet.rb

@@ -122,6 +122,11 @@ module Cosmos
     def set_received_time_fast(received_time)
       @received_time = received_time
       @received_time.freeze if @received_time
+      if @read_conversion_cache
+        synchronize() do
+          @read_conversion_cache.clear
+        end
+      end
     end
 
     # Sets the received count of the packet
@@ -440,11 +445,6 @@ module Cosmos
     # @param value_type (see #read_item)
     # @param buffer (see Structure#write_item)
     def write_item(item, value, value_type = :CONVERTED, buffer = @buffer)
-      if @read_conversion_cache
-        synchronize() do
-          @read_conversion_cache.clear
-        end
-      end
       case value_type
       when :RAW
         super(item, value, value_type, buffer)
@@ -473,6 +473,11 @@ module Cosmos
       else
         raise ArgumentError, "Unknown value type on write: #{value_type}"
       end
+      if @read_conversion_cache
+        synchronize() do
+          @read_conversion_cache.clear
+        end
+      end
     end
 
     # Read an item in the packet by name
@@ -551,9 +556,11 @@ module Cosmos
     # Restore all items in the packet to their default value
     #
     # @param buffer [String] Raw buffer of binary data
-    def restore_defaults(buffer = @buffer)
+    # @param skip_items_names [Array] Array of item names to skip
+    def restore_defaults(buffer = @buffer, skip_item_names = nil)
+      upcase_skip_item_names = skip_item_names.map(&:upcase) if skip_item_names
       @sorted_items.each do |item|
-        write_item(item, item.default, :CONVERTED, buffer)
+        write_item(item, item.default, :CONVERTED, buffer) unless (skip_item_names and upcase_skip_item_names.include?(item.name))
       end
     end
 

data/lib/cosmos/script/scripting.rb

@@ -23,9 +23,11 @@ module Cosmos
     end
 
     def status_bar(message)
-      script_runner = nil
-      ObjectSpace.each_object {|object| if ScriptRunner === object then script_runner = object; break; end}
-      script_runner.script_set_status(message) if script_runner
+      if defined? ScriptRunner
+        script_runner = nil
+        ObjectSpace.each_object {|object| if ScriptRunner === object then script_runner = object; break; end}
+        script_runner.script_set_status(message) if script_runner
+      end
     end
 
     def ask_string(question, blank_or_default = false, password = false)
@@ -553,21 +555,21 @@ module Cosmos
 
     # Require an additional ruby file
     def load_utility(procedure_name)
-      cached = true
+      not_cached = false
       if defined? ScriptRunnerFrame and ScriptRunnerFrame.instance
         saved = ScriptRunnerFrame.instance.use_instrumentation
         begin
           ScriptRunnerFrame.instance.use_instrumentation = false
-          cached = start(procedure_name)
+          not_cached = start(procedure_name)
         ensure
           ScriptRunnerFrame.instance.use_instrumentation = saved
         end
       else # Just call start
-        cached = start(procedure_name)
+        not_cached = start(procedure_name)
       end
       # Return whether we had to load and instrument this file, i.e. it was not cached
       # This is designed to match the behavior of Ruby's require and load keywords
-      !cached
+      not_cached
     end
     alias require_utility load_utility
 

data/lib/cosmos/system/system.rb

@@ -367,6 +367,10 @@ module Cosmos
           dirs.each do |dir_filename|
             if dir_filename[0] != '.'
               if dir_filename == dir_filename.upcase
+                # If any of the targets original directory name matches the
+                # current directory then it must have been already processed by
+                # DECLARE_TARGET so we skip it.
+                next if @targets.select {|name, target| target.original_name == dir_filename }.length > 0
                 if dir_filename == 'SYSTEM'
                   system_found = true
                   next

data/lib/cosmos/system/target.rb

@@ -168,6 +168,8 @@ module Cosmos
       end
       lib_dir = File.join(dir, 'lib')
       Cosmos.add_to_search_path(lib_dir, false) if File.exist?(lib_dir)
+      proc_dir = File.join(dir, 'procedures')
+      Cosmos.add_to_search_path(proc_dir, false) if File.exist?(proc_dir)
       dir
     end
 

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

@@ -500,7 +500,7 @@ module Cosmos
     def get_tlm_buffer(target_name, packet_name)
       packet = System.telemetry.packet(target_name, packet_name)
       return packet.buffer
-    end    
+    end
 
     # Returns all the values (along with their limits state) for a packet.
     #
@@ -977,8 +977,16 @@ module Cosmos
         raise "ERROR: Invalid number of arguments (#{args.length}) passed to #{method_name}()"
       end
 
+      # Build the command
+      begin
+        command = System.commands.build_cmd(target_name, cmd_name, cmd_params, range_check, raw)
+      rescue => e
+        Logger.error e.message
+        raise e
+      end
+
       if hazardous_check
-        hazardous, hazardous_description = System.commands.cmd_hazardous?(target_name, cmd_name, cmd_params)
+        hazardous, hazardous_description = System.commands.cmd_pkt_hazardous?(command)
         if hazardous
           error = HazardousError.new
           error.target_name = target_name
@@ -989,14 +997,6 @@ module Cosmos
         end
       end
 
-      # Build the command
-      begin
-        command = System.commands.build_cmd(target_name, cmd_name, cmd_params, range_check, raw)
-      rescue Exception => e
-        Logger.error e.message
-        raise e
-      end
-
       # Send the command
       @disconnect = false unless defined? @disconnect
       CmdTlmServer.commanding.send_command_to_target(target_name, command) unless @disconnect

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

@@ -154,7 +154,7 @@ module Cosmos
       button = Qt::PushButton.new(button_text)
       if name == ROUTERS
         button.connect(SIGNAL('clicked()')) do
-          if interface.thread
+          if CmdTlmServer.routers.all[interface_name].thread
             Logger.info "User disconnecting router #{interface_name}"
             CmdTlmServer.instance.disconnect_router(interface_name)
           else
@@ -164,7 +164,7 @@ module Cosmos
         end
       else
         button.connect(SIGNAL('clicked()')) do
-          if interface.thread
+          if CmdTlmServer.interfaces.all[interface_name].thread
             Logger.info "User disconnecting interface #{interface_name}"
             CmdTlmServer.instance.disconnect_interface(interface_name)
           else

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

@@ -222,7 +222,7 @@ module Cosmos
         if err
           Logger.info "Connection Lost for #{@interface.name}: #{err.formatted(false, false)}"
           case err
-          when Errno::ECONNABORTED, Errno::ECONNRESET, Errno::ETIMEDOUT, Errno::EBADF, IOError
+          when Errno::ECONNABORTED, Errno::ECONNRESET, Errno::ETIMEDOUT, Errno::EBADF, Errno::ENOTSOCK, IOError
             # Do not write an exception file for these extremely common cases
           else
             Logger.error err.formatted

data/lib/cosmos/tools/packet_viewer/packet_viewer.rb

@@ -125,8 +125,6 @@ module Cosmos
       @derived_last_action.connect(SIGNAL('triggered()')) { update_tlm_items() }
 
       @formatted_tlm_units_action = Qt::Action.new(tr('Formatted Telemetry With &Units'), self)
-      @formatted_tlm_units_keyseq = Qt::KeySequence.new(tr('Ctrl+U'))
-      @formatted_tlm_units_action.shortcut  = @formatted_tlm_units_keyseq
       @formatted_tlm_units_action.statusTip = tr('Formatted Telemetry with Units')
       @formatted_tlm_units_action.setCheckable(true)
       @formatted_tlm_units_action.setChecked(true)
@@ -136,8 +134,6 @@ module Cosmos
       end
 
       @formatted_tlm_action = Qt::Action.new(tr('&Formatted Telemetry'), self)
-      @formatted_tlm_keyseq = Qt::KeySequence.new(tr('Ctrl+F'))
-      @formatted_tlm_action.shortcut  = @formatted_tlm_keyseq
       @formatted_tlm_action.statusTip = tr('Formatted Telemetry')
       @formatted_tlm_action.setCheckable(true)
       @formatted_tlm_action.connect(SIGNAL('triggered()')) do
@@ -146,8 +142,6 @@ module Cosmos
       end
 
       @normal_tlm_action = Qt::Action.new(tr('Normal &Converted Telemetry'), self)
-      @normal_tlm_keyseq = Qt::KeySequence.new(tr('Ctrl+C'))
-      @normal_tlm_action.shortcut  = @normal_tlm_keyseq
       @normal_tlm_action.statusTip = tr('Normal Converted Telemetry')
       @normal_tlm_action.setCheckable(true)
       @normal_tlm_action.connect(SIGNAL('triggered()')) do
@@ -156,8 +150,6 @@ module Cosmos
       end
 
       @raw_tlm_action = Qt::Action.new(tr('&Raw Telemetry'), self)
-      @raw_tlm_keyseq = Qt::KeySequence.new(tr('Ctrl+A'))
-      @raw_tlm_action.shortcut  = @raw_tlm_keyseq
       @raw_tlm_action.statusTip = tr('Raw Unprocessed Telemetry')
       @raw_tlm_action.setCheckable(true)
       @raw_tlm_action.connect(SIGNAL('triggered()')) do
@@ -319,9 +311,6 @@ module Cosmos
         return
       end
 
-      # Clear Status Bar
-      statusBar.clearMessage()
-
       # Update Telemetry Description
       @description.text = ""
       @packets.each do |name, description|
@@ -351,7 +340,7 @@ module Cosmos
               @derived_row += 1
             end
           end
-	  tlm_items.concat(derived) # Tack the derived onto the end
+          tlm_items.concat(derived) # Tack the derived onto the end
         else
           System.telemetry.items(target_name, packet_name).each do |item|
             tlm_items << [item.name, item.states, item.description]
@@ -411,15 +400,16 @@ module Cosmos
 
             begin
               tlm_items = get_tlm_packet(target_name || '', packet_name || '', @mode)
-            rescue DRb::DRbConnError => err
+            rescue DRb::DRbConnError => error
               Qt.execute_in_main_thread(true) do
                 statusBar.showMessage(tr("Error Connecting to Command and Telemetry Server"))
               end
               tlm_items = nil
               update_needed = false
-            rescue RuntimeError => err
+            rescue RuntimeError => error
               Qt.execute_in_main_thread(true) do
-                statusBar.showMessage(tr("Packet #{target_name} #{packet_name} does not exist"))
+                Cosmos.handle_critical_exception(error)
+                statusBar.showMessage(tr("Packet #{target_name} #{packet_name} Error: #{error}"))
               end
               tlm_items = nil
               update_needed = true

data/lib/cosmos/tools/script_runner/script_runner.rb

@@ -79,12 +79,6 @@ module Cosmos
       @file_new.statusTip = tr('Start a new script')
       @file_new.connect(SIGNAL('triggered()')) { file_new() }
 
-      @file_open = Qt::Action.new(Cosmos.get_icon('open.png'), tr('&Open'), self)
-      @file_open_keyseq = Qt::KeySequence.new(tr('Ctrl+O'))
-      @file_open.shortcut  = @file_open_keyseq
-      @file_open.statusTip = tr('Open a script')
-      @file_open.connect(SIGNAL('triggered()')) { file_open() }
-
       @file_close = Qt::Action.new(tr('&Close'), self)
       @file_close_keyseq = Qt::KeySequence.new(tr('Ctrl+W'))
       @file_close.shortcut  = @file_close_keyseq
@@ -249,18 +243,43 @@ module Cosmos
 
     def initialize_menus
       # File Menu
-      file_menu = menuBar.addMenu(tr('&File'))
-      file_menu.addAction(@file_new)
-      file_menu.addAction(@file_open)
-      file_menu.addAction(@file_close)
-      file_menu.addAction(@file_reload)
-      file_menu.addSeparator()
-      file_menu.addAction(@file_save)
-      file_menu.addAction(@file_save_as)
-      file_menu.addSeparator()
-      file_menu.addAction(@file_options)
-      file_menu.addSeparator()
-      file_menu.addAction(@exit_action)
+      @file_menu = menuBar.addMenu(tr('&File'))
+      @file_menu.addAction(@file_new)
+
+      open_action = Qt::Action.new(self)
+      open_action.shortcut = Qt::KeySequence.new(tr('Ctrl+O'))
+      open_action.connect(SIGNAL('triggered()')) { file_open(@procedure_dir) }
+      self.addAction(open_action)
+
+      @file_open = @file_menu.addMenu(tr('&Open'))
+      @file_open.setIcon(Cosmos.get_icon('open.png'))
+      System.paths['PROCEDURES'].each do |path|
+        next unless File.exist? path
+        path_context = path.split('/')[-2..-1].join('/')
+        action = Qt::Action.new(tr(path_context), self)
+        action.statusTip = "Open #{path}"
+        action.connect(SIGNAL('triggered()')) { file_open(path) }
+        @file_open.addAction(action)
+      end
+      @file_open.addSeparator()
+      System.targets.each do |target_name, target|
+        proc_dir = File.join(target.dir, 'procedures')
+        next unless File.exist? proc_dir
+        action = Qt::Action.new(tr("#{target_name}/procedures"), self)
+        action.statusTip = "Open #{proc_dir}"
+        action.connect(SIGNAL('triggered()')) { file_open(File.join(target.dir, 'procedures')) }
+        @file_open.addAction(action)
+      end
+
+      @file_menu.addAction(@file_close)
+      @file_menu.addAction(@file_reload)
+      @file_menu.addSeparator()
+      @file_menu.addAction(@file_save)
+      @file_menu.addAction(@file_save_as)
+      @file_menu.addSeparator()
+      @file_menu.addAction(@file_options)
+      @file_menu.addSeparator()
+      @file_menu.addAction(@exit_action)
 
       # Edit Menu
       mode_menu = menuBar.addMenu(tr('&Edit'))
@@ -366,8 +385,8 @@ module Cosmos
 
     # File->Open
     def file_open(filename = nil)
-      if !filename
-        filename = Qt::FileDialog.getOpenFileName(self, "Select Script", @procedure_dir)
+      if File.directory?(filename)
+        filename = Qt::FileDialog.getOpenFileName(self, "Select Script", filename)
       end
       unless filename.nil? || filename.empty?
         # If the user opens a file we already have open
@@ -905,7 +924,7 @@ module Cosmos
           option_parser, options = create_default_options()
           options.width = 750
           options.height = 600
-          options.title = "Script Runner : Untitiled"
+          options.title = "Script Runner : Untitled"
           options.auto_size = false
           options.config_file = "script_runner.txt"
           options.server_config_file = CmdTlmServer::DEFAULT_CONFIG_FILE

data/lib/cosmos/tools/table_manager/table.rb

@@ -37,9 +37,9 @@ module Cosmos
 
     # Calls define_item in Packet but also incrememts @num_columns
     # if this table is a REPEATING table.
-    def create_param(name, bit_offset, bit_size, type, description, range, default, display_type, editable)
+    def create_param(name, bit_offset, bit_size, type, description, range, default, display_type, editable, endianness)
       @num_columns += 1 if @type == :TWO_DIMENSIONAL
-      item = define_item(name, bit_offset, bit_size, type)
+      item = define_item(name, bit_offset, bit_size, type, nil, endianness)
       item.description = description
       item.range = range
       item.default = default

data/lib/cosmos/tools/table_manager/table_config.rb

@@ -115,8 +115,8 @@ module Cosmos
             end
 
           when 'PARAMETER'
-            usage = "PARAMETER <Parameter Name> <Parameter Description> <Data Type> <Bit Size> <Display Type> <Minimum Value> <Maximum Value> <Default Value - Only in ONE_DIMENTIONAL>"
-            parser.verify_num_parameters(7, 8, usage)
+            usage = "PARAMETER <Parameter Name> <Parameter Description> <Data Type> <Bit Size> <Display Type> <Minimum Value> <Maximum Value> <Default Value (Only in ONE_DIMENTIONAL)> <ENDIANNESS (Optional)>"
+            parser.verify_num_parameters(6, 9, usage)
             finish_parameter()
             if @current_table
               @current_table.num_rows += 1
@@ -134,21 +134,29 @@ module Cosmos
                 type = read_item_type(parameters[2])
                 bit_size = parameters[3].to_i
                 display_type, editable = read_display_type(parameters[4], type)
-                if type == :BLOCK
+                if type == :STRING || type == :BLOCK
                   range = nil
-                elsif type == :STRING
-                  range = convert_to_range(parameters[5], parameters[6], :UINT, 0)
                 else
                   range = convert_to_range(parameters[5], parameters[6], type, bit_size)
                 end
                 if @current_table.type == :ONE_DIMENSIONAL
-                  default = convert_to_type(parameters[7], type)
+                  if type == :STRING || type == :BLOCK
+                    default = convert_to_type(parameters[5], type)
+                  else
+                    default = convert_to_type(parameters[7], type)
+                  end
                 else # TWO_DIMENSIONAL defaults are set by the DEFAULT keyword
                   default = 0
                 end
 
-                @current_parameter = @current_table.create_param(name, @cur_bit_offset,
-                  bit_size, type, description, range, default, display_type, editable)
+                endianness = parameters[-1].to_s.upcase.intern
+                if endianness != :BIG_ENDIAN && endianness != :LITTLE_ENDIAN
+                  endianness = @current_table.default_endianness
+                end
+
+                @current_parameter = @current_table.create_param(
+                  name, @cur_bit_offset, bit_size, type, description,
+                  range, default, display_type, editable, endianness)
                 @cur_bit_offset += parameters[3].to_i
               rescue ArgumentError => err
                 raise parser.error("#{err.message} with #{keyword}.\nUSAGE: #{usage}")
@@ -664,8 +672,8 @@ module Cosmos
     # backwards and that the minimum and maximum values make sense for the given
     # type and bit_size. For example, a maximum of 256 doesn't make sense for a UINT8.
     def convert_to_range(min, max, type, bit_size)
-      min = convert_to_type(min, type)
-      max = convert_to_type(max, type)
+      min = ConfigParser.handle_defined_constants(min.convert_to_value, type, bit_size)
+      max = ConfigParser.handle_defined_constants(max.convert_to_value, type, bit_size)
       range = min..max
 
       # First check for backwards ranges

data/lib/cosmos/tools/table_manager/table_manager.rb

@@ -463,7 +463,7 @@ module Cosmos
           title = @currently_displayed_table_name
         end
         dialog.set_title_and_text("#{title} Hex Dump", str)
-        dialog.set_size(550, 400)
+        dialog.set_size(650, 400)
         dialog.exec
         dialog.dispose
       rescue => err
@@ -804,6 +804,10 @@ module Cosmos
 
             when :STATE
               x = item_def.states[gui_table.item(r,c).text]
+
+            when :STRING, :NONE
+              x = gui_table.item(r,c).text
+
             end
 
             # If there is a read conversion we first read the converted value before writing.
@@ -820,9 +824,7 @@ module Cosmos
           # in this case force the range check to fail
           rescue => error
             text = gui_table.item(r,c).text
-            result << "Error saving #{item_def.name} value of #{text} due to #{error.message}.\nReverting to default of #{item_def.default}\n"
-            table.write(item_def.name, item_def.default)
-            update_gui_item(name, table, item_def, r, c)
+            result << "Error saving #{item_def.name} value of '#{text}' due to #{error.message}.\nDefault value is '#{item_def.default}'\n"
           end
         end # end each table column
       end # end each table row
@@ -889,64 +891,64 @@ module Cosmos
       gui_table = @gui_tables[table_name]
 
       case item_def.display_type
-        when :STATE
-          item = Qt::TableWidgetItem.new
-          item.setData(Qt::DisplayRole, Qt::Variant.new(table_def.read(item_def.name)))
-          gui_table.setItem(table_row, table_column, item)
-          if item_def.editable
-            gui_table.item(table_row, table_column).setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsEditable)
-          else
-            gui_table.item(table_row, table_column).setFlags(Qt::NoItemFlags)
-          end
+      when :STATE
+        item = Qt::TableWidgetItem.new
+        item.setData(Qt::DisplayRole, Qt::Variant.new(table_def.read(item_def.name)))
+        gui_table.setItem(table_row, table_column, item)
+        if item_def.editable
+          gui_table.item(table_row, table_column).setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsEditable)
+        else
+          gui_table.item(table_row, table_column).setFlags(Qt::NoItemFlags)
+        end
 
-        when :CHECK
-          gui_table.setItem(table_row, table_column, Qt::TableWidgetItem.new(table_def.read(item_def.name)))
-          # the ItemData will be 0 for unchecked (corresponds with min value),
-          # and 1 for checked (corresponds with max value)
-          if table_def.read(item_def.name) == item_def.range.begin
-            gui_table.item(table_row, table_column).setCheckState(Qt::Unchecked)
-          else
-            gui_table.item(table_row, table_column).setCheckState(Qt::Checked)
-          end
-          if item_def.editable
-            gui_table.item(table_row, table_column).setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsUserCheckable)
-          else
-            gui_table.item(table_row, table_column).setFlags(Qt::NoItemFlags)
-          end
+      when :CHECK
+        gui_table.setItem(table_row, table_column, Qt::TableWidgetItem.new(table_def.read(item_def.name)))
+        # the ItemData will be 0 for unchecked (corresponds with min value),
+        # and 1 for checked (corresponds with max value)
+        if table_def.read(item_def.name) == item_def.range.begin
+          gui_table.item(table_row, table_column).setCheckState(Qt::Unchecked)
+        else
+          gui_table.item(table_row, table_column).setCheckState(Qt::Checked)
+        end
+        if item_def.editable
+          gui_table.item(table_row, table_column).setFlags(Qt::ItemIsSelectable | Qt::ItemIsEnabled | Qt::ItemIsUserCheckable)
+        else
+          gui_table.item(table_row, table_column).setFlags(Qt::NoItemFlags)
+        end
 
-        when :DEC
-          gui_table.setItem(table_row, table_column, Qt::TableWidgetItem.new(tr(table_def.read(item_def.name).to_s)))
-          if item_def.editable
-            gui_table.item(table_row, table_column).setFlags(Qt::ItemIsSelectable | Qt::ItemIsEditable | Qt::ItemIsEnabled)
-          else
-            gui_table.item(table_row, table_column).setFlags(Qt::NoItemFlags)
-          end
+      when :STRING, :NONE, :DEC
+        gui_table.setItem(table_row, table_column, Qt::TableWidgetItem.new(tr(table_def.read(item_def.name).to_s)))
+        if item_def.editable
+          gui_table.item(table_row, table_column).setFlags(Qt::ItemIsSelectable | Qt::ItemIsEditable | Qt::ItemIsEnabled)
+        else
+          gui_table.item(table_row, table_column).setFlags(Qt::NoItemFlags)
+        end
 
-        when :HEX
-          case item_def.bit_size
-            when 8
-              x = sprintf("%02X", table_def.read(item_def.name).to_s)
-              # if the number was negative x will have .. and possibly another
-              # F in the string which we remove by taking the last 4 digits
-              x = /\w{2}$/.match(x)[0]
-            when 16
-              x = sprintf("%04X", table_def.read(item_def.name).to_s)
-              # if the number was negative x will have .. and possibly another
-              # F in the string which we remove by taking the last 4 digits
-              x = /\w{4}$/.match(x)[0]
-            else
-              x = sprintf("%08X", table_def.read(item_def.name).to_s)
-              # if the number was negative x will have .. and possibly another
-              # F in the string which we remove by taking the last 8 digits
-              x = /\w{8}$/.match(x)[0]
-          end
-          x = Integer("0x#{x}") # convert to Integer
-          gui_table.setItem(table_row, table_column, Qt::TableWidgetItem.new(tr("0x%X" % x)))
-          if item_def.editable
-            gui_table.item(table_row, table_column).setFlags(Qt::ItemIsSelectable | Qt::ItemIsEditable | Qt::ItemIsEnabled)
-          else
-            gui_table.item(table_row, table_column).setFlags(Qt::NoItemFlags)
-          end
+      when :HEX
+        case item_def.bit_size
+        when 8
+          x = sprintf("%02X", table_def.read(item_def.name).to_s)
+          # if the number was negative x will have .. and possibly another
+          # F in the string which we remove by taking the last 4 digits
+          x = /\w{2}$/.match(x)[0]
+        when 16
+          x = sprintf("%04X", table_def.read(item_def.name).to_s)
+          # if the number was negative x will have .. and possibly another
+          # F in the string which we remove by taking the last 4 digits
+          x = /\w{4}$/.match(x)[0]
+        else
+          x = sprintf("%08X", table_def.read(item_def.name).to_s)
+          # if the number was negative x will have .. and possibly another
+          # F in the string which we remove by taking the last 8 digits
+          x = /\w{8}$/.match(x)[0]
+        end
+        x = Integer("0x#{x}") # convert to Integer
+        gui_table.setItem(table_row, table_column, Qt::TableWidgetItem.new(tr("0x%X" % x)))
+        if item_def.editable
+          gui_table.item(table_row, table_column).setFlags(Qt::ItemIsSelectable | Qt::ItemIsEditable | Qt::ItemIsEnabled)
+        else
+          gui_table.item(table_row, table_column).setFlags(Qt::NoItemFlags)
+        end
       end
     end