From 633c08dc1b09123305ae26936342753816727c35 Mon Sep 17 00:00:00 2001
From: hcasperfu <hcasperfu@lbl.gov>
Date: Wed, 9 Aug 2023 12:56:36 -0700
Subject: [PATCH 1/3] set totPum.nin = numPum

---
 .../DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo        | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo b/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
index 5d676935407..e407b8b22c9 100644
--- a/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
+++ b/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
@@ -80,7 +80,7 @@ model ChilledWaterPumpSpeed
     annotation (Placement(transformation(extent={{-80,-50},{-60,-30}})));
   Modelica.Blocks.Math.RealToBoolean twoPum(threshold=1.5) "Two pumps are on"
     annotation (Placement(transformation(extent={{10,-60},{-10,-40}})));
-  Modelica.Blocks.Math.Sum totPum(nin=2) "Total number of pumps on"
+  Modelica.Blocks.Math.Sum totPum(final nin=numPum) "Total number of pumps on"
     annotation (Placement(transformation(extent={{42,-60},{22,-40}})));
   Modelica.Blocks.Interfaces.BooleanInput on
     "On signal of the plant"

From 5b09336aa00d6fed870cc12c64331939922208e7 Mon Sep 17 00:00:00 2001
From: hcasperfu <hcasperfu@lbl.gov>
Date: Wed, 9 Aug 2023 14:00:50 -0700
Subject: [PATCH 2/3] updated documentation and release notes

---
 .../Cooling/Controls/ChilledWaterPumpSpeed.mo    | 16 +++++++++++++++-
 Buildings/package.mo                             | 12 ++++++++++++
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo b/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
index e407b8b22c9..fefb1d4d9ed 100644
--- a/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
+++ b/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
@@ -161,6 +161,20 @@ equation
       revisions="<html>
 <ul>
 <li>
+August 9, 2023, by Hongxiang Fu:<br/>
+Restricted this block to a two-pump configuration as intended.
+<ul>
+<li>
+Set <code>final totPum.nin = numPum</code>.
+</li>
+<li>
+Corrected the \"up to two pumps\" language in documentation.
+</li>
+</ul>
+This is for
+<a href=\"https://github.com/lbl-srg/modelica-buildings/issues/3470\">#3470</a>.
+</li>
+<li>
 December 30, 2022, by Kathryn Hinkelman:<br/>
 Added an <code>on</code> input for a plant-level override to turn pumps off.
 </li>
@@ -193,7 +207,7 @@ reference point from the demand side. The output <code>y</code> is a vector
 of pump speeds.
 </p>
 <p>
-The model currently only supports the control of up to two variable speed pumps.
+The model currently only supports the control of two variable speed pumps.
 </p>
 </html>"));
 end ChilledWaterPumpSpeed;
diff --git a/Buildings/package.mo b/Buildings/package.mo
index 3bd95c72214..85bd2f49ceb 100644
--- a/Buildings/package.mo
+++ b/Buildings/package.mo
@@ -387,6 +387,18 @@ have been <b style=\"color:blue\">improved</b> in a
 <b style=\"color:blue\">backward compatible</b> way:
 </p>
 <table class=\"releaseTable\" summary=\"summary\" border=\"1\" cellspacing=\"0\" cellpadding=\"2\" style=\"border-collapse:collapse;\">
+<tr><td colspan=\"2\"><b>Buildings.Experimental.DHC.Plants.Cooling</b>
+    </td>
+</tr>
+<tr><td valign=\"top\">Buildings.Experimental.DHC.Plants.Cooling.Controls.ChilledWaterPumpSpeed
+    </td>
+    <td valign=\"top\">Set <code>final totPum.nin = numPum</code> so that this block is restricted
+                       to a two-pump configuration as intended.
+                       Also corrected the \"up to two pumps\" language in documentation.<br/>
+                       This is for
+                       <a href=\"https://github.com/lbl-srg/modelica-buildings/issues/3470\">issue 3470</a>.
+    </td>
+</tr>    
 <tr><td colspan=\"2\"><b>Buildings.ThermalZones.Detailed.Validation.BESTEST</b>
     </td>
 </tr>

From 8d42c7cbbb3d023347db912e0f8d7f1d5702f92d Mon Sep 17 00:00:00 2001
From: Michael Wetter <mwetter@lbl.gov>
Date: Fri, 11 Aug 2023 17:10:27 -0700
Subject: [PATCH 3/3] Corrected description

---
 .../DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo        | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo b/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
index fefb1d4d9ed..8b0ca7afddb 100644
--- a/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
+++ b/Buildings/Experimental/DHC/Plants/Cooling/Controls/ChilledWaterPumpSpeed.mo
@@ -1,6 +1,6 @@
 within Buildings.Experimental.DHC.Plants.Cooling.Controls;
 model ChilledWaterPumpSpeed
-  "Controller for up to two headered variable speed chilled water pumps"
+  "Controller for two headered variable speed chilled water pumps"
   extends Modelica.Blocks.Icons.Block;
   parameter Modelica.Units.SI.PressureDifference dpSetPoi(displayUnit="Pa")
     "Pressure difference setpoint";

Buildings.Experimental.DHC.Plants.Cooling +
Buildings.Experimental.DHC.Plants.Cooling.Controls.ChilledWaterPumpSpeed +	Set `final totPum.nin = numPum` so that this block is restricted + to a two-pump configuration as intended. + Also corrected the \"up to two pumps\" language in documentation. + This is for + issue 3470. +
Buildings.ThermalZones.Detailed.Validation.BESTEST