-
Notifications
You must be signed in to change notification settings - Fork 157
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
remove various warnings issued by Spawn #2220
Comments
@kbenne : With Spawn-0.1.0-3ec0a1fa6e-Linux.tar.gz, the warning about the humidity is still there:
Is this checked and passes on your CI test? |
The model I was testing didn't show that one. I think it was the small office reference building that I was using in Chicago. Anyway, I'll see what I can do. This one might not be addressed by release. I do not have automated tests to capture the warning count so CI is blissfully unaware. |
This will be corrected after Buildings 8.0.0 release. |
This is still an open issue, and now there is another warning about the W/person. The log is
|
There have been a couple of long standing issues related to Spawn's zone air moisture calculation, which has manifested as several EnergyPlus warnings. The warning has been a symptom of a more fundamental issue that is resolved in this commit. 1. For zones connected to Modelica, there is a zone air humidity ratio input exposed by the FMU, but this input was not connected to the EnergyPlus state variables. 2. For zones *not* connected to Modelica, the zone air humidity ratio needs to be updated internally, just as we do for the zone air temperature of unconnected zones. This has not happening for zone air humidity ratio. This commit addresses both items 1 and 2, and as a result for the test model "Buildings.ThermalZones.EnergyPlus.Examples.SingleFamilyHouse.AirHeating", the warnings related to zone air humidity are cleared up. ref lbl-srg/modelica-buildings#2220
A recent change to make the epfmi library unique caused the possiblity of the library name starting with a numeric character, which triggers a warning. This change is to prefix the library name with "epfmi_" ref lbl-srg/modelica-buildings#2956 ref lbl-srg/modelica-buildings#2220
@mwetter I've fixed two reoccurring warnings.
I've been specifically using the Buildings.ThermalZones.EnergyPlus.Examples.SingleFamilyHouse.AirHeating model as a test case, but if you know of other models that are generating different warnings the let me know. I think this clears up the common warnings that we are seeing. Note that item 2 is significant beyond simply fixing a warning. The warning was a symptom of the underlying problem that I was not updating the moisture balance correctly. |
Simulating
Buildings.ThermalZones.EnergyPlus.Validation.Schedule.EquipmentSchedule
in Dymola leads to the warnings below.This issue is to remove them.
The output is from 753a530 (the current master).
The text was updated successfully, but these errors were encountered: