openstudio-workflow 2.2.0 → 2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d01f7cb6363d0859380d1052b968b81abcce6d547374045ea311e51118f2493
-  data.tar.gz: e3cde5096a65771a86c4bfb04d190704c2b9a034be3cbad8f5e664d693126a52
+  metadata.gz: 03716fad02e514aa60fbb2e3a14dd2c50286fe2621219f63137f4e1bb77afcb9
+  data.tar.gz: 525a28b94195445ccf890f784898f9868226aea11d0137d112223c2ea0abd099
 SHA512:
-  metadata.gz: aca10e06acecf1f2b0f9875c44b165da9a5165b514fcf76802d16bb86c321338a57bd359f481df6ce8c451ab1a313fd71dc55433be09e063e4e600d9f7753d66
-  data.tar.gz: 202cafbb5c9e04d45e840bdb0938533312a57b4e0c50b3be0ee0d563c16c54d478de54eec72f5eedea270a48ba7d4ce54593191fc9b2ca3e89230c001f53e96f
+  metadata.gz: bc22ddb2820ea5155ce5308a49cfc405f5bdfa43a6b54fbac46ad8486deadb0f8bd866305566dc6fc70d65dbe71feb2c249fd6c0fa94623427f27b630b4c6fc7
+  data.tar.gz: de613152849f97dd568c2f91a3411cb0c3a0f37d48f3e86a3fd37e7922d074e058eab6515e82f4359edb3e175baa718a91c8cbf6cbe1cc414afacee0d9c3253a

data/CHANGELOG.md

@@ -1,6 +1,13 @@
 OpenStudio::Workflow Change Log
 ==================================
 
+Version 2.2.1
+-------------
+* Fixes [#4150](https://github.com/NREL/OpenStudio/issues/4150) LoadError changes current working directory in CLI
+* Add skip option to not zip up datapoint results
+* Update measure tester gem which upgrades Rubocop to 1.15
+* Update styles to v4 based on new version of Rubocop
+
 Version 2.2.0
 -------------
 * Minimum Ruby version upgraded to 2.7.0

data/lib/openstudio/workflow/adapters/input/local.rb

@@ -192,6 +192,24 @@ module OpenStudio
           return default
         end
 
+        def skip_zip_results(user_options, default)
+          # user option trumps all others
+          return user_options[:skip_zip_results] if user_options[:skip_zip_results]
+
+          # try to read from OSW
+          if @run_options && !@run_options.empty?
+            if @run_options.get.respond_to?(:skipZipResults)
+              return @run_options.get.skipZipResults
+            else
+              if @workflow[:run_options]
+                return @workflow[:run_options][:skip_zip_results]
+              end
+            end
+          end
+
+          return default
+        end
+
         def preserve_run_dir(user_options, default)
           # user option trumps all others
           return user_options[:preserve_run_dir] if user_options[:preserve_run_dir]

data/lib/openstudio/workflow/adapters/output/local.rb

@@ -136,8 +136,10 @@ module OpenStudio
 
         # Write the results of the workflow to the filesystem
         #
-        def communicate_results(directory, results)
-          zip_results(directory)
+        def communicate_results(directory, results, skip_zip_results)
+          if !skip_zip_results
+            zip_results(directory)
+          end
 
           if results.is_a? Hash
             # DLM: don't we want this in the results zip?

data/lib/openstudio/workflow/adapters/output/socket.rb

@@ -55,10 +55,6 @@ module OpenStudio
           @socket.write("Started\n")
         end
 
-        def communicate_results(directory, results)
-          super
-        end
-
         def communicate_complete
           super
           @socket.write("Complete\n")
@@ -75,7 +71,7 @@ module OpenStudio
 
         def communicate_transition(message, type, options = {})
           super
-          @socket.write(message + "\n")
+          @socket.write("#{message}\n")
         end
 
         def communicate_energyplus_stdout(line, options = {})

data/lib/openstudio/workflow/adapters/output/web.rb

@@ -47,22 +47,6 @@ module OpenStudio
           raise 'The required :url option was not passed to the web output adapter' unless options[:url]
         end
 
-        def communicate_started
-          super
-        end
-
-        def communicate_results(directory, results)
-          super
-        end
-
-        def communicate_complete
-          super
-        end
-
-        def communicate_failure
-          super
-        end
-
         def communicate_objective_function(objectives, options = {})
           super
         end

data/lib/openstudio/workflow/adapters/output_adapter.rb

@@ -113,19 +113,18 @@ module OpenStudio
             end
 
             # skip x-large directory
-            if File.size?(file)
-              next if File.size?(file) >= 15000000
+            if File.size?(file) && (File.size?(file) >= 15000000)
+              next
             end
+
             add_directory_to_zip(zf, file, directory)
           else
             next if File.extname(file) =~ /\.rb.*/
             next if File.extname(file) =~ /\.zip.*/
 
             # skip large non-osm/idf files
-            if File.size(file)
-              if File.size(file) >= 100000000
-                next unless File.extname(file) == '.osm' || File.extname(file) == '.idf'
-              end
+            if File.size(file) && (File.size(file) >= 100000000) && !(File.extname(file) == '.osm' || File.extname(file) == '.idf')
+              next
             end
 
             zip_file_to_add = file.gsub("#{directory}/", '')

data/lib/openstudio/workflow/jobs/run_initialization.rb

@@ -181,7 +181,7 @@ class RunInitialization < OpenStudio::Workflow::Job
 
     unless weather_path.empty?
       weather_path = weather_path.get
-      @logger.debug 'Searching for weather file #{weather_path}'
+      @logger.debug "Searching for weather file #{weather_path}"
 
       weather_full_path = workflow_json.findFile(weather_path)
       if weather_full_path.empty?

data/lib/openstudio/workflow/jobs/run_reporting_measures.rb

@@ -69,10 +69,8 @@ class RunReportingMeasures < OpenStudio::Workflow::Job
       workflow = nil
       if File.exist? @registry[:osw_path]
         workflow = ::JSON.parse(File.read(@registry[:osw_path]), symbolize_names: true)
-        if !workflow.nil?
-          if !workflow[:urbanopt].nil?
-            @registry.register(:urbanopt) { workflow[:urbanopt] }
-          end
+        if !workflow.nil? && !workflow[:urbanopt].nil?
+          @registry.register(:urbanopt) { workflow[:urbanopt] }
         end
       end
     end

data/lib/openstudio/workflow/run.rb

@@ -48,12 +48,7 @@ module OpenStudio
     class Run
       attr_accessor :registry
 
-      attr_reader :options
-      attr_reader :input_adapter
-      attr_reader :output_adapter
-      attr_reader :final_message
-      attr_reader :job_results
-      attr_reader :current_state
+      attr_reader :options, :input_adapter, :output_adapter, :final_message, :job_results, :current_state
 
       # Define the default set of jobs. Note that the states of :queued of :finished need to exist for all job arrays.
       #
@@ -90,6 +85,7 @@ module OpenStudio
       # @option user_options [Hash] :debug Print debugging messages, overrides OSW option if set, defaults to false
       # @option user_options [Hash] :energyplus_path Specifies path to energyplus executable, defaults to empty
       # @option user_options [Hash] :fast Speeds up workflow by skipping steps not needed for running simulations, defaults to false
+      # @option user_options [Hash] :skip_zip_results Skips creating the data_point.zip file. Setting to `true` can cause issues with workflows expecting .zip files to signal completion (e.g., OpenStudio Analysis Framework), defaults to false
       # @option user_options [Hash] :jobs Simulation workflow, overrides jobs in OSW if set, defaults to default_jobs
       # @option user_options [Hash] :output_adapter Output adapter to use, overrides output adapter in OSW if set, defaults to local adapter
       # @option user_options [Hash] :preserve_run_dir Prevents run directory from being cleaned prior to run, overrides OSW option if set, defaults to false - DLM, Deprecate
@@ -158,15 +154,13 @@ module OpenStudio
         end
 
         # By default blow away the entire run directory every time and recreate it
-        unless @options[:preserve_run_dir]
-          if File.exist?(@registry[:run_dir])
-            # logger is not initialized yet (it needs run dir to exist for log)
-            puts "Removing existing run directory #{@registry[:run_dir]}" if @options[:debug]
-
-            # DLM: this is dangerous, we are calling rm_rf on a user entered directory, need to check this first
-            # TODO: Echoing Dan's comment
-            FileUtils.rm_rf(@registry[:run_dir])
-          end
+        if !@options[:preserve_run_dir] && File.exist?(@registry[:run_dir])
+          # logger is not initialized yet (it needs run dir to exist for log)
+          puts "Removing existing run directory #{@registry[:run_dir]}" if @options[:debug]
+
+          # DLM: this is dangerous, we are calling rm_rf on a user entered directory, need to check this first
+          # TODO: Echoing Dan's comment
+          FileUtils.rm_rf(@registry[:run_dir])
         end
         FileUtils.mkdir_p(@registry[:run_dir])
 
@@ -176,7 +170,7 @@ module OpenStudio
         else
           # don't create these files unless we want to use them
           # DLM: TODO, make sure that run.log will be closed later
-          @options[:targets] = [STDOUT, File.open(File.join(@registry[:run_dir], 'run.log'), 'a')]
+          @options[:targets] = [$stdout, File.open(File.join(@registry[:run_dir], 'run.log'), 'a')]
         end
 
         @registry.register(:log_targets) { @options[:targets] }
@@ -204,6 +198,7 @@ module OpenStudio
         @options[:verify_osw] = @input_adapter.verify_osw(user_options, true)
         @options[:weather_file] = @input_adapter.weather_file(user_options, nil)
         @options[:fast] = @input_adapter.fast(user_options, false)
+        @options[:skip_zip_results] = @input_adapter.skip_zip_results(user_options, false)
 
         openstudio_dir = 'unknown'
         begin
@@ -236,7 +231,7 @@ module OpenStudio
 
           if !@options[:fast]
             @logger.info 'Finished workflow - communicating results and zipping files'
-            @output_adapter.communicate_results(@registry[:run_dir], @registry[:results])
+            @output_adapter.communicate_results(@registry[:run_dir], @registry[:results], @options[:skip_zip_results])
           end
         rescue StandardError => e
           @logger.info "Error occurred during running with #{e.message}"

data/lib/openstudio/workflow/util/energyplus.rb

@@ -125,7 +125,7 @@ module OpenStudio
         # @return [Void]
         #
         def call_energyplus(run_directory, energyplus_path = nil, output_adapter = nil, logger = nil, workflow_json = nil)
-          logger ||= ::Logger.new(STDOUT) unless logger
+          logger ||= ::Logger.new($stdout) unless logger
 
           current_dir = Dir.pwd
           energyplus_path ||= find_energyplus
@@ -231,7 +231,7 @@ module OpenStudio
           end
 
           # merge in monthly reports
-          EnergyPlus.monthly_report_idf_text.split(/^[\s]*$/).each do |object|
+          EnergyPlus.monthly_report_idf_text.split(/^\s*$/).each do |object|
             object = object.strip
             next if object.empty?
 
@@ -278,11 +278,9 @@ module OpenStudio
           allowed_objects << 'Meter:CustomDecrement'
           allowed_objects << 'EnergyManagementSystem:OutputVariable'
 
-          if allowed_objects.include?(idd_object.name)
-            unless check_for_object(workspace, idf_object, idd_object.type)
-              workspace.addObject(idf_object)
-              num_added += 1
-            end
+          if allowed_objects.include?(idd_object.name) && !check_for_object(workspace, idf_object, idd_object.type)
+            workspace.addObject(idf_object)
+            num_added += 1
           end
 
           allowed_unique_objects = []
@@ -293,15 +291,13 @@ module OpenStudio
           # OutputControl:ReportingTolerances # not allowed
           # Output:SQLite # not allowed
 
-          if allowed_unique_objects.include?(idf_object.iddObject.name)
-            if idf_object.iddObject.name == 'Output:Table:SummaryReports'
-              summary_reports = workspace.getObjectsByType(idf_object.iddObject.type)
-              if summary_reports.empty?
-                workspace.addObject(idf_object)
-                num_added += 1
-              else
-                merge_output_table_summary_reports(summary_reports[0], idf_object)
-              end
+          if allowed_unique_objects.include?(idf_object.iddObject.name) && (idf_object.iddObject.name == 'Output:Table:SummaryReports')
+            summary_reports = workspace.getObjectsByType(idf_object.iddObject.type)
+            if summary_reports.empty?
+              workspace.addObject(idf_object)
+              num_added += 1
+            else
+              merge_output_table_summary_reports(summary_reports[0], idf_object)
             end
           end
 

data/lib/openstudio/workflow/util/measure.rb

@@ -342,6 +342,8 @@ module OpenStudio
             result = nil
             begin
               load measure_path.to_s
+              # load.c in ruby can result in changing dir to root / so preserve cwd here. happens in openstudio cli 
+              Dir.chdir measure_run_dir 
               measure_object = Object.const_get(class_name).new
             rescue => e
 

data/lib/openstudio/workflow/util/model.rb

@@ -95,7 +95,7 @@ module OpenStudio
         # @todo (rhorsey) rescue errors here
         #
         def translate_to_energyplus(model, logger = nil)
-          logger ||= ::Logger.new(STDOUT)
+          logger ||= ::Logger.new($stdout)
           logger.info 'Translate object to EnergyPlus IDF in preparation for EnergyPlus'
           a = ::Time.now
           # ensure objects exist for reporting purposes

data/lib/openstudio/workflow/util/post_process.rb

@@ -138,10 +138,10 @@ module OpenStudio
         #
         def rename_hash_keys(hash, logger)
           # @todo should we log the name changes?
-          regex = %r{[|!@#\$%^&\*\(\)\{\}\\\[\];:'",<.>/?\+=]+}
+          regex = %r{[|!@#$%^&*()\{\}\\\[\];:'",<.>/?+=]+}
 
           rename_keys = lambda do |h|
-            if Hash === h
+            if h.is_a?(Hash)
               h.each_key do |key|
                 if key.to_s =~ regex
                   logger.warn "Renaming result key '#{key}' to remove invalid characters"

data/lib/openstudio/workflow/util/weather_file.rb

@@ -55,7 +55,7 @@ module OpenStudio
         #
         def get_weather_file(directory, wf, wf_search_array, model, logger = nil)
           # TODO: this logic needs some updating, weather file should come from current model, found using search paths
-          logger ||= ::Logger.new(STDOUT) unless logger
+          logger ||= ::Logger.new($stdout) unless logger
           if wf
             weather_file = get_weather_file_from_fs(directory, wf, wf_search_array, logger)
             raise 'Could not locate the weather file in the filesystem. Please see the log' if weather_file == false

data/lib/openstudio/workflow/version.rb

@@ -37,6 +37,6 @@
 
 module OpenStudio
   module Workflow
-    VERSION = '2.2.0'.freeze # Suffixes must have periods (not dashes)
+    VERSION = '2.2.1' # Suffixes must have periods (not dashes)
   end
 end

data/lib/openstudio/workflow_json.rb

@@ -89,9 +89,7 @@ class WorkflowStepResultValue_Shim
     @type = type
   end
 
-  attr_reader :name
-
-  attr_reader :value
+  attr_reader :name, :value
 
   def variantType
     @type
@@ -162,7 +160,7 @@ class WorkflowStepResult_Shim
 
   def setStepResult(step_result)
     @result[:step_result] = step_result
- end
+  end
 end
 
 # WorkflowStep_Shim provides a shim interface to the WorkflowStep class in OpenStudio 2.X when running in OpenStudio 1.X

data/lib/openstudio/workflow_runner.rb

@@ -65,14 +65,12 @@ class WorkflowRunner < OpenStudio::Ruleset::OSRunner
     ::Time.now.utc.strftime('%Y%m%dT%H%M%SZ')
   end
 
-  attr_reader :datapoint
+  attr_reader :datapoint, :analysis
 
   def setDatapoint(datapoint)
     @datapoint = datapoint
   end
 
-  attr_reader :analysis
-
   def setAnalysis(analysis)
     @analysis = analysis
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openstudio-workflow
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.2.1
 platform: ruby
 authors:
 - Nicholas Long
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-15 00:00:00.000000000 Z
+date: 2021-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: builder
@@ -95,20 +95,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 2.8.0
+- !ruby/object:Gem::Dependency
+  name: openstudio_measure_tester
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.3.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.3.1
 - !ruby/object:Gem::Dependency
   name: openstudio-standards
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.12
+        version: 0.2.14
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.12
+        version: 0.2.14
 - !ruby/object:Gem::Dependency
   name: parallel
   requirement: !ruby/object:Gem::Requirement
@@ -157,14 +171,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.3'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.3'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -179,34 +193,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.9'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.54.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.54.0
-- !ruby/object:Gem::Dependency
-  name: rubocop-checkstyle_formatter
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.4.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.4.0
 description: Run OpenStudio based measures and simulations using EnergyPlus
 email:
 - nicholas.long@nrel.gov