Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

EMS Actuators Override Python Actuators Silently #8084

Closed
3 tasks done
mitchute opened this issue Jun 16, 2020 · 0 comments · Fixed by #8139
Closed
3 tasks done

EMS Actuators Override Python Actuators Silently #8084

mitchute opened this issue Jun 16, 2020 · 0 comments · Fixed by #8139
Assignees
@jmarrec

Comments

@mitchute
Copy link
Collaborator

mitchute commented Jun 16, 2020 •
edited by jmarrec
Loading

If a Python plugin actuator is defined and the same actuator is defined in the IDF file using regular EMS, the Python actuator is overridden silently and the simulation continues with no warning or error messages. EnergyPlus should detect this duplication and raise a warning to the user.

Details

Some additional details for this issue (if relevant):

  • Platform (Mac 10.14.5)
  • Version of EnergyPlus (v9.4.0 fbcde7d)
  • Unmethours link or helpdesk ticket number - None

Checklist

Add to this list or remove from it as applicable. This is a simple templated set of guidelines.
@mitchute mitchute changed the title EMS Actuators Override Python Actuator Silently EMS Actuators Override Python Actuators Silently Jun 16, 2020
@jmarrec jmarrec self-assigned this Jul 7, 2020
jmarrec added a commit that referenced this issue Jul 8, 2020
@jmarrec
Add test for #8084
0131258
@jmarrec jmarrec mentioned this issue Jul 8, 2020
Merged
20 tasks
jmarrec added a commit that referenced this issue Jul 8, 2020
@jmarrec
Add test for #8084
89d3298
jmarrec added a commit that referenced this issue Jul 8, 2020
@jmarrec
Fix #8084 - Warn if an EMS:Actuator is already setup when calling get…
9006ad8 
…ActuatorHandle
Myoldmopar added a commit that referenced this issue Sep 7, 2020
@Myoldmopar
Merge pull request #8139 from NREL/8084_Python_EMS_Override
986165d 
Fix #8084 - EMS Actuators Override Python Actuators Silently
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@jmarrec jmarrec

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #8084 - EMS Actuators Override Python Actuators Silently NREL/EnergyPlus
2 participants
@mitchute @jmarrec