Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Openstudio Forward translator failing to translate ElectricLoadCenter:Distribution Inverter name attribute #4175

Closed
chrisbalbach opened this issue Dec 29, 2020 · 5 comments
Assignees

Comments

@chrisbalbach
Copy link

Issue overview

Models

Current Behavior

OpenStudio models using PVWatts Inverter Objects fail with the following error: (from eplusout.err)

** Severe ** ElectPowerLoadCenter constructor ElectricLoadCenter:Distribution="ELECTRIC LOAD CENTER DISTRIBUTION 1", invalid entry.
** ~~~ ** Inverter Name is blank, but buss type requires inverter.
** Fatal ** ElectPowerLoadCenter constructor Preceding errors terminate program

Expected Behavior

Model should simulate as expected, without EnergyPlus failure

Steps to Reproduce

See attached files - .

  1. in.osm file with properly specified ElectricLoadCenterDistribution object attributes
  2. resulting in.idf file that is produced

Possible Solution

It appears the forward translator is not properly handling the inverter assignment defined in the ElectricLoadCenterDistribution object.

Details

N/A

Environment

Some additional details about your environment for this issue (if relevant):

  • OpenStudio v3.1.0 (via OS App v1.1.0)
  • Windows 64 Bit OS

Context

Not able to properly model ElectricLoadCenterDistribution interaction with PV Watts generation scenarios w/o potentially resorting to a EnergyPlus measure as a work around.

Not able to properly model ElectricLoadCenterDistribution interaction with PV Watts + Simple Battery Storage either, which is a real drag. #

Proper operation of these features are needed to provide feedback for Grid Interactive Efficient Building Operation scenarios

@chrisbalbach chrisbalbach added the Triage Issue needs to be assessed and labeled, further information on reported might be needed label Dec 29, 2020
@chrisbalbach
Copy link
Author

data_point.zip

@chrisbalbach
Copy link
Author

inspection of in.osm reveals data model appears complete. Inspection of in.idf file reveals data model is incomplete.
Manual creation of inverter object (using idfeditor tool) and manual assignment of inverter object to Distribution Center object allows idf file to properly simulate.

@shorowit
Copy link
Contributor

@chrisbalbach FWIW, we use PVWatts in the OS-HPXML workflow without a problem, see relevant code here. Maybe that will show you some way to avoid an EnergyPlus measure.

@tijcolem tijcolem added severity - Minor Bug and removed Triage Issue needs to be assessed and labeled, further information on reported might be needed labels Jan 8, 2021
@joseph-robertson
Copy link
Collaborator

@chrisbalbach Couple of questions/observations about your osm.

Looks like you have 4 ElectricLoadCenter:Distribution objects in your osm:

  • 1 and 3 have generators, but no inverters assigned (therefore I wouldn't expect inverters attached to these distribution objects in the idf)
  • 2 and 4 have inverters, but no generators assigned (these distribution objects don't appear to be translated to idf; I'm investigating this)

Perhaps you're intending to have exactly 2 ElectricLoadCenter:Distribution objects in your model (combine distribution objects 1+2 and objects 3+4)?

@chrisbalbach
Copy link
Author

My failed file appears to be due to an artifact of how the measure that inserts the ElectricLoad Distribution objects.

Mea Culpa (operator error)...

Closing issue.

joseph-robertson added a commit that referenced this issue Jan 29, 2021
Addresses #4175, issues with ft for ElectricLoadCenter:Distribution
jmarrec added a commit that referenced this issue Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants