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

Reasoning the result of /examples/python/testcase1.py #582

Open
kefeimo opened this issue Oct 18, 2023 · 1 comment
Open

Reasoning the result of /examples/python/testcase1.py #582

kefeimo opened this issue Oct 18, 2023 · 1 comment
Assignees

Comments

@kefeimo
Copy link
Contributor

kefeimo commented Oct 18, 2023

Description

When running the example https://github.com/ibpsa/project1-boptest/blob/master/examples/python/testcase1.py and plot the result, it gives the following plot.

image

From GET/measurements

{"message":"Queried the measurements successfully.","payload":{"CO2RooAir_y":{"Description":"Zone air CO2 concentration","Maximum":null,"Minimum":null,"Unit":"ppm"},"PHea_y":{"Description":"Heater power","Maximum":null,"Minimum":null,"Unit":"W"},"TRooAir_y":{"Description":"Zone air temperature","Maximum":null,"Minimum":null,"Unit":"K"}},"status":200}

from @rlutes "When the PID signal (oveAct_u) goes negative the heating power (PHea_y) should be zero but it is not. It appears there is something in the model taking the absolute value of the PID signal controlling the heater inside the model."

Expected outcome

Would like to understand the simulation result and confirm it is behaving as expected.

@dhblum
Copy link
Collaborator

dhblum commented Nov 3, 2023

The performance is as expected, but the documentation of the model and example controller could be improved:

  • The HVAC device controlled is actually better characterized as a "heater/cooler" rather than just "heater" since it can provide heating and cooling.
  • The Power use of the device is calculated within the model simply as the absolute value of the heating or cooling thermal power delivered divided by an efficiency.
  • The "PID" controller implemented in python is really just a "P" controller, as it only implements proportional control.

I suggest to address these confusions by appropriately fixing naming conventions.

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

3 participants