First time here? Check our help page!
1

Error in Running AEDG Office HVAC VAV CHW in PAT

Hello all,

The measure AEDG Office HVAC CHW keeps failing in PAT for me. I tried applying the measure in OpenStudio to see if it is just the measure, and it might be the case. the measure fails and I get the following message:

image description

image description

image description

I tried deleting the measure from my local library and then re-downloading it for PAT and the openstudio test, but got the same result. Failed to apply the measure, and a failed run in PAT. Sorry in advance if pictures do not post correctly. I am running Openstudio 2.6.

R1cardo's avatar
51
R1cardo
asked 2019-08-28 17:44:39 -0500
MatthewSteen's avatar
10.1k
MatthewSteen
updated 2019-08-29 12:55:51 -0500
edit flag offensive 0 remove flag close merge delete

Comments

add a comment see more comments

1 Answer

3

There is a variable name mismatch in the measure in lines 225, 230, 236, and 238. The variable for the .each loop is 'plantloop' (lower case l) and the variables in lines 230 and 238 are 'plantLoop' (upper case L):

# remove plant loops
plantLoops.each do |plantloop|
  # get the demand components and see if water use connection, then save it
  # notify user with info statement if supply side of plant loop had heat exchanger for refrigeration
  usedForSHWorRefrigeration = false
  plantLoop.demandComponents.each do |comp| # AP code to check your comments above
    if comp.to_WaterUseConnections.is_initialized || comp.to_CoilWaterHeatingDesuperheater.is_initialized
      usedForSHWorRefrigeration = true
    end
  end
  if usedForSHWorRefrigeration == false
    plantloop.remove
  else
    runner.registerWarning("#{plantLoop.name} is used for SHW or refrigeration heat reclaim.  Loop will not be deleted")
  end
end

@david-goldwasser or I will push a fix to the measure on BCL and comment here once it is updated. You can also manually change the variable names to be consistent for a short term fix.

EDIT: measure is updated on BCL

mdahlhausen's avatar
9.5k
mdahlhausen
answered 2019-08-29 10:38:05 -0500, updated 2019-08-29 11:26:42 -0500
edit flag offensive 0 remove flag delete link

Comments

Thanks! Should they all be 'plantloop' with the lower case l?

R1cardo's avatar R1cardo (2019-08-29 10:59:59 -0500) edit
1

doesn't matter, they just need to be consistent

mdahlhausen's avatar mdahlhausen (2019-08-29 11:10:17 -0500) edit

Thank you. Saw the Edit. Re downloaded the measure, deleted the old one and recreated. Will report back if something goes wrong.

R1cardo's avatar R1cardo (2019-08-29 11:39:09 -0500) edit
1

Updated measure is live on BCL. The date last updated may not be correct, but I downloaded a copy to confirm the fix.

David Goldwasser's avatar David Goldwasser (2019-08-29 11:47:38 -0500) edit
add a comment see more comments