openstudio-standards 0.8.3 → 0.8.5.rc1

data/lib/openstudio-standards/thermal_zone/{thermal_zone.rb → information.rb}

@@ -75,7 +75,7 @@ module OpenstudioStandards
         space.spaceInfiltrationDesignFlowRates.each do |infil|
           if infil.designFlowRate.is_initialized
             is_vest = true
-            OpenStudio.logFree(OpenStudio::Info, 'OpenstudioStandards::ThermalZone', "For #{thermal_zone.name}: This zone is considered a vestibule.")
+            OpenStudio.logFree(OpenStudio::Info, 'openstudio.standards.ThermalZone', "For #{thermal_zone.name}: This zone is considered a vestibule.")
             break
           end
         end
@@ -175,7 +175,7 @@ module OpenstudioStandards
             htd = true
           end
         else
-          OpenStudio.logFree(OpenStudio::Debug, 'OpenstudioStandards::ThermalZone', "Zone #{thermal_zone.name} used an unknown schedule type for the heating setpoint; assuming heated.")
+          OpenStudio.logFree(OpenStudio::Debug, 'openstudio.standards.ThermalZone', "Zone #{thermal_zone.name} used an unknown schedule type for the heating setpoint; assuming heated.")
           htd = true
         end
       end
@@ -211,7 +211,7 @@ module OpenstudioStandards
               htd = true
             end
           else
-            OpenStudio.logFree(OpenStudio::Debug, 'OpenstudioStandards::ThermalZone', "Zone #{thermal_zone.name} used an unknown schedule type for the heating setpoint; assuming heated.")
+            OpenStudio.logFree(OpenStudio::Debug, 'openstudio.standards.ThermalZone', "Zone #{thermal_zone.name} used an unknown schedule type for the heating setpoint; assuming heated.")
             htd = true
           end
         end
@@ -316,7 +316,7 @@ module OpenstudioStandards
             cld = true
           end
         else
-          OpenStudio.logFree(OpenStudio::Debug, 'OpenstudioStandards::ThermalZone', "Zone #{thermal_zone.name} used an unknown schedule type for the cooling setpoint; assuming cooled.")
+          OpenStudio.logFree(OpenStudio::Debug, 'openstudio.standards.ThermalZone', "Zone #{thermal_zone.name} used an unknown schedule type for the cooling setpoint; assuming cooled.")
           cld = true
         end
       end
@@ -352,7 +352,7 @@ module OpenstudioStandards
               cld = true
             end
           else
-            OpenStudio.logFree(OpenStudio::Debug, 'OpenstudioStandards::ThermalZone', "Zone #{thermal_zone.name} used an unknown schedule type for the cooling setpoint; assuming cooled.")
+            OpenStudio.logFree(OpenStudio::Debug, 'openstudio.standards.ThermalZone', "Zone #{thermal_zone.name} used an unknown schedule type for the cooling setpoint; assuming cooled.")
             cld = true
           end
         end
@@ -384,7 +384,7 @@ module OpenstudioStandards
     def self.thermal_zone_electric_heat?(thermal_zone)
       # error if HVACComponent heating fuels method is not available
       if thermal_zone.model.version < OpenStudio::VersionString.new('3.6.0')
-        OpenStudio.logFree(OpenStudio::Error, 'OpenstudioStandards::ThermalZone', 'Required HVACComponent method .heatingFuelTypes is not available in pre-OpenStudio 3.6.0 versions. Use a more recent version of OpenStudio.')
+        OpenStudio.logFree(OpenStudio::Error, 'openstudio.standards.ThermalZone', 'Required HVACComponent method .heatingFuelTypes is not available in pre-OpenStudio 3.6.0 versions. Use a more recent version of OpenStudio.')
       end
 
       # Get an array of the heating fuels used by the zone
@@ -404,7 +404,7 @@ module OpenstudioStandards
     def self.thermal_zone_fossil_heat?(thermal_zone)
       # error if HVACComponent heating fuels method is not available
       if thermal_zone.model.version < OpenStudio::VersionString.new('3.6.0')
-        OpenStudio.logFree(OpenStudio::Error, 'OpenstudioStandards::ThermalZone', 'Required HVACComponent method .heatingFuelTypes is not available in pre-OpenStudio 3.6.0 versions. Use a more recent version of OpenStudio.')
+        OpenStudio.logFree(OpenStudio::Error, 'openstudio.standards.ThermalZone', 'Required HVACComponent method .heatingFuelTypes is not available in pre-OpenStudio 3.6.0 versions. Use a more recent version of OpenStudio.')
       end
 
       is_fossil = false
@@ -438,7 +438,7 @@ module OpenstudioStandards
     def self.thermal_zone_district_heat?(thermal_zone)
       # error if HVACComponent heating fuels method is not available
       if thermal_zone.model.version < OpenStudio::VersionString.new('3.6.0')
-        OpenStudio.logFree(OpenStudio::Error, 'OpenstudioStandards::ThermalZone', 'Required HVACComponent method .heatingFuelTypes is not available in pre-OpenStudio 3.6.0 versions. Use a more recent version of OpenStudio.')
+        OpenStudio.logFree(OpenStudio::Error, 'openstudio.standards.ThermalZone', 'Required HVACComponent method .heatingFuelTypes is not available in pre-OpenStudio 3.6.0 versions. Use a more recent version of OpenStudio.')
       end
 
       is_district = false
@@ -467,39 +467,6 @@ module OpenstudioStandards
       return is_mixed
     end
 
-    # Adds a thermostat that heats the space to 0 F and cools to 120 F.
-    # These numbers are outside of the threshold that is considered heated
-    # or cooled by thermal_zone_cooled?() and thermal_zone_heated?()
-    #
-    # @param thermal_zone [OpenStudio::Model::ThermalZone] OpenStudio ThermalZone object
-    # @return [Boolean] returns true if successful, false if not
-    def self.thermal_zone_add_unconditioned_thermostat(thermal_zone)
-      # Heated to 0F (below thermal_zone_heated?(thermal_zone)  threshold)
-      htg_t_f = 0
-      htg_t_c = OpenStudio.convert(htg_t_f, 'F', 'C').get
-      htg_stpt_sch = OpenStudio::Model::ScheduleRuleset.new(thermal_zone.model)
-      htg_stpt_sch.setName('Unconditioned Minimal Heating')
-      htg_stpt_sch.defaultDaySchedule.setName('Unconditioned Minimal Heating Default')
-      htg_stpt_sch.defaultDaySchedule.addValue(OpenStudio::Time.new(0, 24, 0, 0), htg_t_c)
-
-      # Cooled to 120F (above thermal_zone_cooled?(thermal_zone)  threshold)
-      clg_t_f = 120
-      clg_t_c = OpenStudio.convert(clg_t_f, 'F', 'C').get
-      clg_stpt_sch = OpenStudio::Model::ScheduleRuleset.new(thermal_zone.model)
-      clg_stpt_sch.setName('Unconditioned Minimal Heating')
-      clg_stpt_sch.defaultDaySchedule.setName('Unconditioned Minimal Heating Default')
-      clg_stpt_sch.defaultDaySchedule.addValue(OpenStudio::Time.new(0, 24, 0, 0), clg_t_c)
-
-      # Thermostat
-      thermostat = OpenStudio::Model::ThermostatSetpointDualSetpoint.new(thermal_zone.model)
-      thermostat.setName("#{thermal_zone.name} Unconditioned Thermostat")
-      thermostat.setHeatingSetpointTemperatureSchedule(htg_stpt_sch)
-      thermostat.setCoolingSetpointTemperatureSchedule(clg_stpt_sch)
-      thermal_zone.setThermostatSetpointDualSetpoint(thermostat)
-
-      return true
-    end
-
     # Determine the design internal load (W) for this zone without space multipliers.
     # This include People, Lights, Electric Equipment, and Gas Equipment in all spaces in this zone.
     # It assumes 100% of the wattage is converted to heat, and that the design peak schedule value is 1 (100%).
@@ -569,7 +536,7 @@ module OpenstudioStandards
       building_type = building_type_areas.key(building_type_areas.values.max)
 
       if building_type.nil?
-        OpenStudio.logFree(OpenStudio::Info, 'OpenstudioStandards::ThermalZone', "Thermal zone #{thermal_zone.name} does not have standards building type.")
+        OpenStudio.logFree(OpenStudio::Info, 'openstudio.standards.ThermalZone', "Thermal zone #{thermal_zone.name} does not have standards building type.")
       end
 
       return building_type
@@ -687,7 +654,7 @@ module OpenstudioStandards
       # Convert to cfm
       tot_oa_flow_rate_cfm = OpenStudio.convert(tot_oa_flow_rate, 'm^3/s', 'cfm').get
 
-      OpenStudio.logFree(OpenStudio::Debug, 'openstudio.Standards.ThermalZone', "For #{thermal_zone.name}, design min OA = #{tot_oa_flow_rate_cfm.round} cfm.")
+      OpenStudio.logFree(OpenStudio::Debug, 'openstudio.standards.ThermalZone', "For #{thermal_zone.name}, design min OA = #{tot_oa_flow_rate_cfm.round} cfm.")
 
       return tot_oa_flow_rate
     end
@@ -711,7 +678,7 @@ module OpenstudioStandards
       # Calculate the per-area value
       tot_oa_flow_rate_per_area = tot_oa_flow_rate / sum_floor_area
 
-      # OpenStudio::logFree(OpenStudio::Debug, "openstudio.Standards.Model", "For #{self.name}, OA per area = #{tot_oa_flow_rate_per_area.round(8)} m^3/s*m^2.")
+      # OpenStudio::logFree(OpenStudio::Debug, "openstudio.standards.ThermalZone", "For #{self.name}, OA per area = #{tot_oa_flow_rate_per_area.round(8)} m^3/s*m^2.")
 
       return tot_oa_flow_rate_per_area
     end
@@ -776,7 +743,7 @@ module OpenstudioStandards
           new_dsn_oa.setOutdoorAirFlowRateFractionSchedule(oa_sch)
         end
 
-        OpenStudio.logFree(OpenStudio::Info, 'openstudio.Standards.ThermalZone', "For #{thermal_zone.name}: Converted total ventilation requirements to per-area value.")
+        OpenStudio.logFree(OpenStudio::Info, 'openstudio.standards.ThermalZone', "For #{thermal_zone.name}: Converted total ventilation requirements to per-area value.")
       end
 
       return true

data/lib/openstudio-standards/thermal_zone/thermostat_schedules.rb

@@ -0,0 +1,152 @@
+module OpenstudioStandards
+  # The ThermalZone module provides methods to set thermostats and get information about model thermal zones
+  module ThermalZone
+    # Adds thermostat schedules to thermal zones based on the standards space type
+    # Chooses the thermostat schedule with the most restrictive setpoints (maximum heating, lowest cooling) in the schedule if multiple space types are present
+    #
+    # @param thermal_zones [Array<OpenStudio::Model::ThermalZone>] OpenStudio ThermalZone objects
+    # @return [Boolean] returns true if successful, false if not
+    def self.thermal_zones_set_thermostat_schedules(thermal_zones)
+      # load and return thermostat mapping data
+      thermostat_data = JSON.parse(File.read("#{File.dirname(__FILE__)}/../prototypes/common/data/thermostat_schedule_lookup.json"), symbolize_names: true)
+
+      # std call to access model_add_schedule
+      # @todo refactor once schedule data is separate
+      std = Standard.build('90.1-2013')
+
+      thermal_zones.each do |thermal_zone|
+        # skip plenums
+        if OpenstudioStandards::ThermalZone.thermal_zone_plenum?(thermal_zone)
+          OpenStudio.logFree(OpenStudio::Warn, 'openstudio.standards.ThermalZone', "Thermal Zone '#{thermal_zone.name}' is a plenum. Not adding thermostat schedules.")
+          next
+        end
+
+        # get space types
+        thermal_zone_space_types = []
+        thermal_zone.spaces.each do |space|
+          if space.spaceType.is_initialized
+            thermal_zone_space_types << space.spaceType.get
+          end
+        end
+
+        # check if additional properties set, and if not add it
+        has_space_type = thermal_zone_space_types[0].additionalProperties.hasFeature('standards_space_type')
+        unless has_space_type
+          OpenstudioStandards::SpaceType.set_standards_space_type_additional_properties(thermal_zone.model)
+        end
+
+        # add thermostat schedules for space type
+        zone_thermostats_options = []
+        thermal_zone_space_types.each do |space_type|
+          heating_thermostat_sch_name = nil
+          cooling_thermostat_sch_name = nil
+
+          # get the standards space type
+          space_type_name = space_type.additionalProperties.getFeatureAsString('standards_space_type').get
+
+          # get the thermostat data associated with this space type
+          space_type_data = thermostat_data.select { |h| h[:space_type] == space_type_name }
+
+          # get unique possible heating and cooling setpoint schedules
+          heating_thermostat_schs = space_type_data.map { |h| h[:heating_setpoint_schedule] }.compact.uniq
+          cooling_thermostat_schs = space_type_data.map { |h| h[:cooling_setpoint_schedule] }.compact.uniq
+
+          # check if there is a unique heating and cooling thermostat schedule for this space type
+          if (heating_thermostat_schs.size < 2) && (cooling_thermostat_schs.size < 2)
+            # if so, use it
+            heating_thermostat_sch_name = heating_thermostat_schs[0]
+            cooling_thermostat_sch_name = cooling_thermostat_schs[0]
+          else
+            # if not, get the heating and cooling thermostat schedule by standards building type
+            if space_type.standardsBuildingType.is_initialized
+              # select down to building type
+              space_type_data = space_type_data.select { |h| h[:standards_building_type] == space_type.standardsBuildingType.get }
+              if space_type_data.empty?
+                OpenStudio.logFree(OpenStudio::Warn, 'openstudio.standards.ThermalZone', "No thermostat schedule data is available for space type '#{space_type.name} with standards space type #{space_type_name} and standards building type #{space_type.standardsBuildingType.get}. Unable to create thermostat schedules.")
+                next
+              else
+                heating_thermostat_sch_name = space_type_data[0][:heating_setpoint_schedule]
+                cooling_thermostat_sch_name = space_type_data[0][:cooling_setpoint_schedule]
+              end
+            else
+              # unable to find standards building type
+              OpenStudio.logFree(OpenStudio::Warn, 'openstudio.standards.ThermalZone', "Multiple thermostat schedules are available for space type '#{space_type.name} with standards space type #{space_type_name} depending on building type, but building type is not specified. Using a default schedule for this space type.")
+              space_type_data = thermostat_data.find { |h| h[:space_type] == space_type_name }
+              heating_thermostat_sch_name = space_type_data[:heating_setpoint_schedule]
+              cooling_thermostat_sch_name = space_type_data[:cooling_setpoint_schedule]
+            end
+          end
+
+          # make thermostat
+          thermostat = space_type.model.getThermostatSetpointDualSetpointByName("#{space_type.name} Thermostat")
+          if thermostat.is_initialized
+            OpenStudio.logFree(OpenStudio::Debug, 'openstudio.standards.ThermalZone', "#{space_type.name} thermostat already in model.")
+            thermostat = thermostat.get
+          else
+            thermostat = OpenStudio::Model::ThermostatSetpointDualSetpoint.new(space_type.model)
+            thermostat.setName("#{space_type.name} Thermostat")
+
+            unless heating_thermostat_sch_name.nil?
+              thermostat.setHeatingSetpointTemperatureSchedule(std.model_add_schedule(space_type.model, heating_thermostat_sch_name))
+              OpenStudio.logFree(OpenStudio::Info, 'openstudio.standards.ThermalZone', "#{space_type.name} set heating setpoint schedule to #{heating_thermostat_sch_name}.")
+            end
+
+            unless cooling_thermostat_sch_name.nil?
+              thermostat.setCoolingSetpointTemperatureSchedule(std.model_add_schedule(space_type.model, cooling_thermostat_sch_name))
+              OpenStudio.logFree(OpenStudio::Info, 'openstudio.standards.ThermalZone', "#{space_type.name} set cooling setpoint schedule to #{cooling_thermostat_sch_name}.")
+            end
+          end
+
+          # log the schedules
+          zone_thermostats_options << thermostat
+        end
+
+        # If only one thermostat, use it
+        if zone_thermostats_options.size == 1
+          thermal_zone.setThermostatSetpointDualSetpoint(zone_thermostats_options[0])
+        elsif zone_thermostats_options.size > 1
+          # otherwise find the zone thermostat with the most restrictive setpoints
+          # @todo logic here to pick between options
+          OpenStudio.logFree(OpenStudio::Info, 'openstudio.standards.ThermalZone', "Multiple thermostat options for thermal zone #{thermal_zone.name} depending on space types.")
+          thermal_zone.setThermostatSetpointDualSetpoint(zone_thermostats_options[0])
+        else
+          OpenStudio.logFree(OpenStudio::Info, 'openstudio.standards.ThermalZone', "Unable to find valid thermostat options for thermal zone #{thermal_zone.name} depending on space types.")
+        end
+      end
+
+      return true
+    end
+
+    # Adds thermostat schedules with a 0F heating setpoint and 120F cooling setpoint.
+    # These numbers are outside of the threshold that is considered heated or cooled by thermal_zone_heated? and thermal_zone_cooled?
+    #
+    # @param thermal_zone [OpenStudio::Model::ThermalZone] OpenStudio ThermalZone object
+    # @return [Boolean] returns true if successful, false if not
+    def self.thermal_zone_set_unconditioned_thermostat(thermal_zone)
+      # Heated to 0F (below thermal_zone_heated?(thermal_zone)  threshold)
+      htg_t_f = 0.0
+      htg_t_c = OpenStudio.convert(htg_t_f, 'F', 'C').get
+      htg_stpt_sch = OpenStudio::Model::ScheduleRuleset.new(thermal_zone.model)
+      htg_stpt_sch.setName('Unconditioned Minimal Heating')
+      htg_stpt_sch.defaultDaySchedule.setName('Unconditioned Minimal Heating Default')
+      htg_stpt_sch.defaultDaySchedule.addValue(OpenStudio::Time.new(0, 24, 0, 0), htg_t_c)
+
+      # Cooled to 120F (above thermal_zone_cooled?(thermal_zone)  threshold)
+      clg_t_f = 120.0
+      clg_t_c = OpenStudio.convert(clg_t_f, 'F', 'C').get
+      clg_stpt_sch = OpenStudio::Model::ScheduleRuleset.new(thermal_zone.model)
+      clg_stpt_sch.setName('Unconditioned Minimal Cooling')
+      clg_stpt_sch.defaultDaySchedule.setName('Unconditioned Minimal Cooling Default')
+      clg_stpt_sch.defaultDaySchedule.addValue(OpenStudio::Time.new(0, 24, 0, 0), clg_t_c)
+
+      # Thermostat
+      thermostat = OpenStudio::Model::ThermostatSetpointDualSetpoint.new(thermal_zone.model)
+      thermostat.setName("#{thermal_zone.name} Unconditioned Thermostat")
+      thermostat.setHeatingSetpointTemperatureSchedule(htg_stpt_sch)
+      thermostat.setCoolingSetpointTemperatureSchedule(clg_stpt_sch)
+      thermal_zone.setThermostatSetpointDualSetpoint(thermostat)
+
+      return true
+    end
+  end
+end

data/lib/openstudio-standards/utilities/logging.rb

@@ -4,23 +4,27 @@ $OPENSTUDIO_LOG.setLogLevel(OpenStudio::Debug)
 
 # Log the info, warning, and error messages to a runner.
 #
-# runner @param [Runner] The Measure runner to add the messages to
-# debug @param [Boolean] If true, include the debug messages in the log
-# @return [Runner] The same Measure runner, with messages from the openstudio-standards library added
-def log_messages_to_runner(runner, debug = false)
+# @param runner [OpenStudio::Measure::OSRunner] The Measure runner to add the messages to
+# @param debug [Boolean] If true, include the debug messages in the log
+# @param custom_exclusions [Array<String>] An array of custom strings to exclude from the log
+def log_messages_to_runner(runner, debug = false, custom_exclusions: nil)
+  log_message_exclusions = [
+    'UseWeatherFile', # 'UseWeatherFile' is not yet a supported option for YearDescription
+    'Skipping layer', # Annoying/bogus "Skipping layer" warnings
+    'runmanager', # RunManager messages
+    'setFileExtension', # .ddy extension unexpected
+    'Translator', # Forward translator and geometry translator
+    'Successive data points', # Successive data points (2004-Jan-31 to 2001-Feb-01, ending on line 753) are greater than 1 day apart in EPW file
+    'has multiple parents', # Bogus errors about curves having multiple parents
+    'does not have an Output', # Warning from EMS translation
+    'Prior to OpenStudio 2.6.2, this field was returning a double, it now returns an Optional double' # Warning about OS API change
+  ] + Array(custom_exclusions)
+
   $OPENSTUDIO_LOG.logMessages.each do |msg|
     # DLM: you can filter on log channel here for now
     if /openstudio.*/ =~ msg.logChannel # /openstudio\.model\..*/
       # Skip certain messages that are irrelevant/misleading
-      next if msg.logMessage.include?('UseWeatherFile') || # 'UseWeatherFile' is not yet a supported option for YearDescription
-              msg.logMessage.include?('Skipping layer') || # Annoying/bogus "Skipping layer" warnings
-              msg.logChannel.include?('runmanager') || # RunManager messages
-              msg.logChannel.include?('setFileExtension') || # .ddy extension unexpected
-              msg.logChannel.include?('Translator') || # Forward translator and geometry translator
-              msg.logMessage.include?('Successive data points') || # Successive data points (2004-Jan-31 to 2001-Feb-01, ending on line 753) are greater than 1 day apart in EPW file
-              msg.logMessage.include?('has multiple parents') || # Bogus errors about curves having multiple parents
-              msg.logMessage.include?('does not have an Output') || # Warning from EMS translation
-              msg.logMessage.include?('Prior to OpenStudio 2.6.2, this field was returning a double, it now returns an Optional double') # Warning about OS API change
+      next if log_message_exclusions.any? { |ex| msg.logMessage.include?(ex) }
 
       # Report the message in the correct way
       if msg.logLevel == OpenStudio::Info
@@ -29,7 +33,7 @@ def log_messages_to_runner(runner, debug = false)
         runner.registerWarning("[#{msg.logChannel}] #{msg.logMessage}")
       elsif msg.logLevel == OpenStudio::Error
         runner.registerError("[#{msg.logChannel}] #{msg.logMessage}")
-      elsif msg.logLevel == OpenStudio::Debug && debug
+      elsif debug && msg.logLevel == OpenStudio::Debug
         runner.registerInfo("DEBUG - #{msg.logMessage}")
       end
     end

data/lib/openstudio-standards/version.rb

@@ -13,5 +13,5 @@ module OpenstudioStandards
     end
     return 'git-not-found-on-this-system'
   end
-  VERSION = '0.8.3'.freeze
+  VERSION = '0.8.5.rc1'.freeze
 end

data/lib/openstudio-standards/weather/modify.rb

@@ -79,7 +79,7 @@ module OpenstudioStandards
       end
 
       # set ground temperature shallow values based on .stat file
-      ground_temperature_shallow = OpenStudio::Model::SiteGroundTemperatureShallow.new(model)
+      ground_temperature_shallow = model.getSiteGroundTemperatureShallow
       ground_temperature_shallow.setJanuarySurfaceGroundTemperature(stat_file.monthly_undis_ground_temps_0p5m[0])
       ground_temperature_shallow.setFebruarySurfaceGroundTemperature(stat_file.monthly_undis_ground_temps_0p5m[1])
       ground_temperature_shallow.setMarchSurfaceGroundTemperature(stat_file.monthly_undis_ground_temps_0p5m[2])
@@ -113,7 +113,7 @@ module OpenstudioStandards
       end
 
       # set ground temperature deep values based on .stat file
-      ground_temperature_deep = OpenStudio::Model::SiteGroundTemperatureDeep.new(model)
+      ground_temperature_deep = model.getSiteGroundTemperatureDeep
       ground_temperature_deep.setJanuaryDeepGroundTemperature(stat_file.monthly_undis_ground_temps_4p0m[0])
       ground_temperature_deep.setFebruaryDeepGroundTemperature(stat_file.monthly_undis_ground_temps_4p0m[1])
       ground_temperature_deep.setMarchDeepGroundTemperature(stat_file.monthly_undis_ground_temps_4p0m[2])

data/lib/openstudio-standards/weather/stat_file.rb

@@ -285,7 +285,7 @@ module OpenstudioStandards
 
         # parse design temperatures
         # TMYx stat file format is different than TMY3, cf #1948
-        is_tmyx = @text.include?("Wind Shelter Factor")
+        is_tmyx = @text.include?('Wind Shelter Factor')
         temperature_info = [
           { name: 'Heating Design Temperatures', regex: /Heating(\s*\d+.*)\n/, container: @heating_design_info, size: is_tmyx ? 16 : 15 },
           { name: 'Cooling Design Temperatures', regex: /Cooling(\s*\d+.*)\n/, container: @cooling_design_info, size: 32 },
@@ -294,7 +294,7 @@ module OpenstudioStandards
         ]
         temperature_info.each { |temp_info| parse_design_temp_info(temp_info) }
         if is_tmyx
-          @heating_design_info.pop() # remove the last value, which is the Wind Shelter Factor
+          @heating_design_info.pop # remove the last value, which is the Wind Shelter Factor
           # There is no WBmax in TMYx files, so add a nil
           @extremes_design_info.insert(3, nil)
         end