origen_testers 0.48.3 → 0.49.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3c67bc8fcc8e5ee79fccb2b53ab691ce22f5635811bedcb2cc6cbd6a247d0af7
-  data.tar.gz: 94966a015d097731c5a8320bbe2f0f8d7238b988151af466b213c2017e3f1c53
+  metadata.gz: 44d3544735522514276c1f11198d48015f587645bb434cb5ff5e1cd877dd799c
+  data.tar.gz: 99b3a18e65bd837febf9b9d6cfa1043f447c5cbc02aec73902c35312d2e9f266
 SHA512:
-  metadata.gz: 4628a25c96b619ddc665fc49c958468f9192ea16af3609d04cbb40f27ef714e13ccfba33a111b632e1842f7fb70018eb6665a70f57bfcdc316e829bd1cd1560c
-  data.tar.gz: 7fa99d5a51d669bb845bc529d91dcf69a77c19851313b1cacfca0e1d704f122c23dda8f7495de9893af6ab89103ea2e2648eaa5561cad96ddf2269fdc8f73a4f
+  metadata.gz: 87fb0d43358361dc87fb0db076c75050edfd26c4e4e5c8b62eb21d4794f4a5efe314a019cf7ede6e1d93f31184829e52cf50852c8aac9ffc573bceb97a9eba0e
+  data.tar.gz: d3ff0b325eecffe7a7f61cdc16473fd737b798cfa8dc5c7404ff60831407bdc441bcc158975cf9e34c0110c191287f05d03b056e8f2962bb6fdc7e4e278b37eb

data/config/version.rb

@@ -1,6 +1,6 @@
 module OrigenTesters
   MAJOR = 0
-  MINOR = 48
+  MINOR = 49
   BUGFIX = 3
   DEV = nil
   VERSION = [MAJOR, MINOR, BUGFIX].join(".") + (DEV ? ".pre#{DEV}" : '')

data/lib/origen_testers/api.rb

@@ -11,6 +11,7 @@ module OrigenTesters
     attr_accessor :includes
     attr_accessor :comment_level
     attr_accessor :generating
+
     # Get/Set the overlay style
     #
     # This method changes the way overlay is handled.
@@ -19,6 +20,7 @@ module OrigenTesters
     # @example
     #   tester.overlay_style = :label
     attr_accessor :overlay_style
+
     # Get/Set the capture style
     #
     # This method changes the way tester.store() implements the store
@@ -49,10 +51,18 @@ module OrigenTesters
     end
     alias_method :pattern_extension, :pat_extension
 
+    def comment_char=(val)
+      @comment_char = val
+    end
+
     def comment_char
       @comment_char || '//'
     end
 
+    def program_comment_char=(val)
+      @program_comment_char = val
+    end
+
     def program_comment_char
       @program_comment_char || comment_char
     end

data/lib/origen_testers/atp/flow.rb

@@ -439,6 +439,8 @@ module OrigenTesters::ATP
             children << on_pass(options[:on_pass]) if options[:on_pass]
           end
 
+          children << priority(options[:priority]) if options[:priority]
+
           save_conditions
           n(:test, children)
         end
@@ -581,14 +583,26 @@ module OrigenTesters::ATP
     end
 
     def loop(*args, &block)
-      unless args[0].keys.include?(:from) && args[0].keys.include?(:to) && args[0].keys.include?(:step)
-        fail 'Loop must specify :from, :to, :step'
+      unless args[0].keys.include?(:from) && args[0].keys.include?(:to)
+        fail 'Loop must specify :from, :to'
+      end
+      # assume 1 if :step not provided
+      unless args[0].keys.include?(:step)
+        args[0][:step] = 1
+      end
+      # assume 1 if :test_num_inc not provided
+      unless args[0].keys.include?(:test_num_inc)
+        args[0][:test_num_inc] = 1
+      end
+      # Add node for set of flag to be used for loop
+      unless args[0][:var].nil?
+        set(args[0][:var], 0)
       end
       extract_meta!(options) do
         apply_conditions(options) do
-          # always pass 4-element array to loop node to simplify downstream parser
-          #   last element, 'var', will be nil if not specified by loop call
-          params = [args[0][:from], args[0][:to], args[0][:step], args[0][:var]]
+          # always pass 5-element array to loop node to simplify downstream parser
+          #   element, 'var', will be nil if not specified by loop call
+          params = [args[0][:from], args[0][:to], args[0][:step], args[0][:var], args[0][:test_num_inc]]
 
           node = n(:loop, params)
           node = append_to(node) { block.call }
@@ -601,7 +615,7 @@ module OrigenTesters::ATP
       define_method method do |*args, &block|
         options = args.pop if args.last.is_a?(Hash)
         unless args.size == 2
-          fail "Format for relational operation must match: ':<opertor>(var1, var2)'"
+          fail "Format for relational operation must match: ':<operator>(var1, var2)'"
         end
         n2(method.to_sym, args[0], args[1])
       end unless method_defined?(method)
@@ -810,6 +824,10 @@ module OrigenTesters::ATP
       n1(:id, name)
     end
 
+    def priority(name)
+      n1(:priority, name)
+    end
+
     def on_fail(options = {})
       if options.is_a?(Proc)
         node = n0(:on_fail)

data/lib/origen_testers/charz/profile.rb

@@ -35,6 +35,19 @@ module OrigenTesters
       def attrs_ok?
         return if @quality_check == false
 
+        unless @routines.is_a?(Array)
+          Origen.log.error "Profile #{id}: routines is expected to be of type <Array>, but is instead of type <#{@routines.class}>!"
+          fail
+        end
+
+        # allowing a config for empty routines for usecase of
+        # determining routines on the fly dynamically
+        if @routines.empty? && !@allow_empty_routines
+          Origen.log.error "Profile #{id}: routines array is empty!"
+          Origen.log.warn "If you'd like to enable profile creation without routines, set the profile's @allow_empty_routines attribute to true"
+          fail
+        end
+
         unknown_routines = @routines - @defined_routines
         unless unknown_routines.empty?
           Origen.log.error "Profile #{id}: unknown routines: #{unknown_routines}"

data/lib/origen_testers/flow.rb

@@ -205,6 +205,8 @@ module OrigenTesters
     def active_description
       flow_file = OrigenTesters::Flow.callstack.last
       called_from = caller.find { |l| l =~ /^#{flow_file}:.*/ }
+      # Windows fix - prevent the drive letter in the file name from changing the index of the line_no below
+      called_from.gsub!(flow_file, '')
       desc = nil
       if called_from
         called_from = called_from.split(':')
@@ -257,8 +259,10 @@ module OrigenTesters
       flow_file = OrigenTesters::Flow.callstack.last
       called_from = caller.find { |l| l =~ /^#{flow_file}:.*/ }
       if called_from
+        # Splitting on ':' when file names are included will yield a different index for everything in Windows
+        called_from.gsub!(flow_file, '')
         called_from = called_from.split(':')
-        options[:source_file] = called_from[0]
+        options[:source_file] = flow_file # called_from[0]
         options[:source_line_number] = called_from[1].to_i
       end
     end

data/lib/origen_testers/smartest_based_tester/base/flow.rb

@@ -462,6 +462,41 @@ module OrigenTesters
         alias_method :on_whenever_any, :on_whenever
         alias_method :on_whenever_all, :on_whenever
 
+        def on_loop(node, options = {})
+          # TODO: don't have the SMT8 way to do this yet
+          if smt8?
+            fail 'Flow loop control not yet supported for SMT8!'
+          end
+          start = node.to_a[0]
+          stop = node.to_a[1]
+          step = node.to_a[2]
+          if node.to_a[3].nil?
+            fail 'You must supply a loop variable name!'
+          else
+            var = generate_flag_name(node.to_a[3])
+          end
+          test_num_inc = node.to_a[4]
+          unless smt8?
+            var = "@#{var}"
+          end
+          num = (stop - start) / step + 1
+          # Handle increment/decrement
+          if step < 0
+            compare = '>'
+            incdec = "- #{step * -1}"
+          else
+            compare = '<'
+            incdec = "+ #{step}"
+          end
+          line "for #{var} = #{start}; #{var} #{compare} #{stop + step} ; #{var} = #{var} #{incdec}; do"
+          line "test_number_loop_increment = #{test_num_inc}"
+          line '{'
+          @indent += 1
+          process_all(node.children)
+          @indent -= 1
+          line '}'
+        end
+
         def generate_expr_string(node, options = {})
           return node unless node.respond_to?(:type)
           case node.type

data/lib/origen_testers/smartest_based_tester/base/limits_file.rb

@@ -119,6 +119,7 @@ module OrigenTesters
           o[:test_name] = extract_test_name(node, o)
           o[:test_number] = extract_test_number(node, o)
           o[:limits] = extract_limits(node, o)
+          o[:priority] = extract_priority(node, o)
           o[:test_text] = node.find(:test_text).try(:value)
           if on_fail = node.find(:on_fail)
             if set_result = on_fail.find(:set_result)
@@ -141,7 +142,7 @@ module OrigenTesters
             end
             if smt8?
               if o[:bin_s_num]
-                limits_workbook.add_softbin o[:bin_s_num], name: o[:bin_s_name], bin: o[:bin_h_num]
+                limits_workbook.add_softbin o[:bin_s_num], name: o[:bin_s_name], bin: o[:bin_h_num], priority: o[:priority]
               end
               if o[:bin_h_num]
                 limits_workbook.add_bin o[:bin_h_num], name: o[:bin_h_name]
@@ -217,6 +218,16 @@ module OrigenTesters
           name
         end
 
+        def extract_priority(node, o)
+          test_obj = node.find(:priority).to_a[0]
+          if test_obj.is_a?(Hash)
+            priority = test_obj['Priority']
+          else
+            priority = test_obj.respond_to?(:priority) ? test_obj.priority : test_obj if test_obj
+          end
+          priority
+        end
+
         def extract_test_name(node, o)
           test_obj = node.find(:object).to_a[0]
           if smt8?

data/lib/origen_testers/smartest_based_tester/base.rb

@@ -99,8 +99,10 @@ module OrigenTesters
         @min_repeat_loop = 33
         if smt8?
           @pat_extension = 'pat'
+          @program_comment_char = ['println', '//']
         else
           @pat_extension = 'avc'
+          @program_comment_char = ['print_dl', '//']
         end
         @compress = true
         # @support_repeat_previous = true
@@ -115,6 +117,7 @@ module OrigenTesters
         @overlay_style = :subroutine	# default to use subroutine for overlay
         @capture_style = :hram			# default to use hram for capture
         @overlay_subr = nil
+        @overlay_history = {} # used to track labels, subroutines, digsrc pins used etc
 
         if options[:add_flow_enable]
           self.add_flow_enable = options[:add_flow_enable]
@@ -203,6 +206,12 @@ module OrigenTesters
             when :subroutine, :default
               subroutine_overlay(overlay_str, options)
               ovly_style = :subroutine
+            when :label, :global_label
+              options[:dont_compress] = true
+              unless @overlay_history.key?(overlay_str)
+                cc "#{overlay_str}"
+                @overlay_history[overlay_str] = { is_label: true }
+              end
             when :handshake
               if @delayed_handshake
                 if @delayed_handshake != overlay_str
@@ -458,7 +467,6 @@ module OrigenTesters
         # Ensure the match pins are don't care by default
         pin.dont_care
         options[:pin2].dont_care if options[:pin2]
-
         if !options[:pin2]
           cc "for the #{pin.name.upcase} pin to go #{state.to_s.upcase}"
           match_block(timeout_in_cycles, options) do |match_or_conditions, fail_conditions|

data/program/flow_control.rb

@@ -254,7 +254,6 @@ Flow.create interface: 'OrigenTesters::Test::Interface', flow_name: "Flow Contro
   enable :nvm_minimum_ft, if_enable: "nvm_minimum_room", if_job: :fr
   enable :nvm_minimum_ft, if_enable: "nvm_minimum_cold", if_job: :fc
   disable :nvm_minimum_ft, if_enable: "nvm_minimum_hot", if_job: :fh
-
   log "Test enable words that wrap a lot of tests"
   if_enable :word1 do
     5.times do |i|
@@ -340,6 +339,39 @@ Flow.create interface: 'OrigenTesters::Test::Interface', flow_name: "Flow Contro
     func :test36, on_fail: { render: 'multi_bin;' }, if_flag: :my_flag, number: 51570
   end
 
+  if tester.v93k? && !tester.smt8?
+    log "Tests of flow loop"
+    loop from: 0, to: 5, step: 1, var: '$LOOP_VARIABLE' do
+      func :test_myloop, number: 56000
+    end
+
+    log "Tests of flow loop, no step"
+    loop from: 0, to: 5, var: '$LOOP_VARIABLE' do
+      func :test_myloop2, number: 5610
+    end
+
+    log "Tests of flow loop, non-default test number increment" 
+    loop from: 0, to: 5, var: '$LOOP_VARIABLE', test_num_inc: 2 do
+      func :test_myloop3, number: 56200
+    end
+ 
+    log "Tests of decrementing loop"
+    loop from: 5, to: 2, step: -1, var: '$LOOP_VARIABLE' do
+      func :test_myloop4, number: 56300
+    end
+ 
+    log "Tests of nested flow loop, depth 3"
+    loop from: 0, to: 9, step: 2, var: '$LOOP_VARIABLE1'do
+      loop from: 1, to: 10, step: 1, var: '$LOOP_VARIABLE2' do
+        loop from: 1, to: 5, step: 1, var: '$LOOP_VARIABLE3' do
+          func :test_myloop5, number: 56400
+        end
+      end
+    end
+
+    # Test of skipping variable name not yet ready
+  end
+
   log 'An optimization test case, this should not generate a flag on V93K'
   func :test1, id: :t1a, number: 51580
 

data/program/prb1.rb

@@ -8,6 +8,10 @@ Flow.create interface: 'OrigenTesters::Test::Interface', flow_description: 'Prob
     self.resources_filename = 'prb1'
   end
 
+  # Extra log below not in approved/ folder to show that comments are 
+  # being skipped correctly during the difference checks
+  log 'PRB1 Test Flow Version 00001 - do not copy me to approved!'
+
   import 'components/prb1_main'
 
   import 'test' # import top-level test.rb directly, note that Flow.create options of sub-flow will be ignored!

data/templates/origen_guides/program/flowapi.md.erb

@@ -339,4 +339,37 @@ end
 ~~~
 
 
+#### Flow Loops for V93k (SMT7 only)
+Use flow loop control to permit re-running tests without using additional sequence labels.
+
+~~~ruby
+loop from: 0, to: 5, step: 1, var: '$LOOP_VARIABLE' do
+  func :test_myloop, number: 56000
+end
+~~~
+
+Indicating step value is optional, default is 1.
+
+These loops can also be nested:
+~~~ruby
+loop from: 0, to: 9, step: 2, var: '$LOOP_VARIABLE1'do
+  loop from: 1, to: 10, step: 1, var: '$LOOP_VARIABLE2' do
+    loop from: 1, to: 5, step: 1, var: '$LOOP_VARIABLE3' do
+      func :test_myloop5, number: 56400
+    end
+  end
+end
+~~~
+
+You can also indicate a test number increment if desired (default is 1):
+~~~ruby
+loop from: 0, to: 5, var: '$LOOP_VARIABLE', test_num_inc: 2 do
+    func :test_myloop3, number: 56200
+end
+~~~
+
+Decrementing loops, having `from:` value > `to:` value and using negative `step:`, is also supported.
+ 
+
+
 % end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: origen_testers
 version: !ruby/object:Gem::Version
-  version: 0.48.3
+  version: 0.49.3
 platform: ruby
 authors:
 - Stephen McGinty
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-04-15 00:00:00.000000000 Z
+date: 2021-09-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: origen
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.57.1
+- !ruby/object:Gem::Dependency
+  name: dentaku
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -544,7 +558,6 @@ files:
 - templates/origen_guides/pattern/stil.md.erb
 - templates/origen_guides/pattern/timing.md.erb
 - templates/origen_guides/pattern/ultraflex.md.erb
-- templates/origen_guides/pattern/ultraflex.md.erb~
 - templates/origen_guides/pattern/v93k.md.erb
 - templates/origen_guides/program/charz.md.erb
 - templates/origen_guides/program/code.md.erb
@@ -586,7 +599,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.0.1
 signing_key: 
 specification_version: 4
 summary: This plugin provides Origen tester models to drive ATE type testers like

data/templates/origen_guides/pattern/ultraflex.md.erb~

@@ -1,30 +0,0 @@
-% render "layouts/guides.html" do
-
-This page will be used to document any UltraFLEX-only APIs related to pattern generation,
-however the goal is to have as few of these as possible so that Origen pattern source code can re-target
-automatically to any supported platform.
-
-There are no significant APIs in this category currently, therefore refer to the
-[Common Pattern API](<%= path "guides/pattern/common" %>) which can fully target the UltraFLEX.
-
-### DigSrc
-
-UltraFlex supports <code>:digsrc</code> as a <code>tester.overlay_style</code> set like this:
-
-~~~ruby
-  tester.overlay_style = :digsrc
-~~~
-
-By default Origen will automatically place the digsrc start opcode at the beginning of the resulting pattern
-when overlay is used. In some cases (like when the pattern is used in a pattern set that has already started
-the instrument in a previous pattern, or possibly in svm_patterns) this behavior is undesirable.
-
-The insertion of this start opcode can be disabled by placing the following code **before** any overlay operations
-for a given pin.
-
-~~~ruby
-  tester.digsrc_skip_start :pin_or_group_name if tester.ultraflex?
-  # Overlay operations can happen after this point
-~~~
-
-% end