Question-and-Answer Resource for the Building Energy Modeling Community
Get started with the Help page
Ask Your Question
1

Error in Running AEDG Office HVAC VAV CHW in PAT

asked 2019-08-28 17:44:39 -0500

R1cardo's avatar

updated 2019-08-29 12:55:51 -0500

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.

edit retag flag offensive close merge delete

1 Answer

Sort by ยป oldest newest most voted
3

answered 2019-08-29 10:38:05 -0500

updated 2019-08-29 11:26:42 -0500

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

edit flag offensive delete link more

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

Your Answer

Please start posting anonymously - your entry will be published after you log in or create a new account.

Add Answer

Careers

Question Tools

1 follower

Stats

Asked: 2019-08-28 17:44:39 -0500

Seen: 178 times

Last updated: Aug 29 '19