Skip to content

Commit

Permalink
remove duplicate nZones parameter
Browse files Browse the repository at this point in the history
  • Loading branch information
lucasverleyen committed Dec 23, 2024
1 parent 3443e3f commit 7d69824
Showing 1 changed file with 7 additions and 9 deletions.
16 changes: 7 additions & 9 deletions IDEAS/Templates/Interfaces/BaseClasses/VentilationSystem.mo
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,10 @@ partial model VentilationSystem
constrainedby Modelica.Media.Interfaces.PartialMedium
"Medium in the component"
annotation (choicesAllMatching = true);
parameter Integer nZones(min=1)
"Number of conditioned thermal building zones";
parameter Real[nZones] VZones "Conditioned volumes of the zones";

parameter Modelica.Units.SI.Power[nZones] Q_design=zeros(nZones)
"Design heat loss due to ventilation";//must be calculated depending on the case

Modelica.Blocks.Interfaces.RealInput[nZones] TSensor(
each final quantity="ThermodynamicTemperature",
each unit="K",
Expand Down Expand Up @@ -58,12 +56,12 @@ partial model VentilationSystem
graphics),
Documentation(revisions="<html>
<ul>
<li>
May 22, 2022, by Filip Jorissen:<br/>
Fixed Modelica specification compatibility issue.
See <a href=\"https://github.com/open-ideas/IDEAS/issues/1254\">
#1254</a>
</li>
<li>December 23, 2024, by Lucas Verleyen:<br>
Removed nZones parameter, since it is already declared in PartialSystem, from which this model extends.<br>
See <a href=\"https://github.com/open-ideas/IDEAS/pull/1401\">#1401</a>.</li>
<li>May 22, 2022, by Filip Jorissen:<br>
Fixed Modelica specification compatibility issue.<br>
See <a href=\"https://github.com/open-ideas/IDEAS/issues/1254\">#1254</a>.</li>
</ul>
</html>"));
end VentilationSystem;

0 comments on commit 7d69824

Please sign in to comment.