Rename all_continuous() method to are_all_continuous() and implement caching.

Uses all_continuous property to cache the current value, which is initialized and maintained by add() and set_params(). The are_all_continuous() method only re-evaluates from scratch if the all_continuous property is empty.

Author: rdzman

CHANGES.md

@@ -5,9 +5,10 @@ Change history for MP-Opt-Model
 since version 5.0
 -----------------
 
-#### 4/17/26
-   - Move `mp.opt_model.is_mixed_integer()` logic into new `all_continuous()`
-     method of `mp.sm_variable` and make the former a wrapper.
+#### 4/20/26
+   - Move `mp.opt_model.is_mixed_integer()` logic into new
+     `are_all_continuous()` method of `mp.sm_variable`, and make the former
+     a simple wrapper.
    - Fix bug with parameter caching in `mp.sm_variable`, where calling
      `params()` without requesting the variable type would not cache it,
      so subsequent requests for the variable time incorrectly returned an

docs/src/MP-Opt-Model-manual/MP-Opt-Model-manual.tex

@@ -6742,7 +6742,7 @@ \subsubsection*{New Features}
 \item Support for the new PDHG LP solver in \gurobi{}.
 \item New methods:
     \begin{itemize}
-    \item \code{mp.sm\_variable.all\_continuous()} method returns true if all variables are continuous, false otherwise.
+    \item \code{mp.sm\_variable.are\_all\_continuous()} method returns true if all variables are continuous, false otherwise.
     \end{itemize}
 \item New functions:
     \begin{itemize}
@@ -6752,7 +6752,7 @@ \subsubsection*{New Features}
 
 \subsubsection*{Bugs Fixed}
 \begin{itemize}
-\item Calling the \code{params()} method of \code{mp.sm\_variable} without less than 4 output arguments would prevent the variable type from being assembled and cached, resulting in an incorrect empty string being returned on subsequent calls with the fourth output argument present.
+\item Calling the \code{params()} method of \code{mp.sm\_variable} with less than 4 output arguments would prevent the variable type from being assembled and cached, resulting in an incorrect empty string being returned on subsequent calls with the fourth output argument present.
 \item
     \emph{Thanks to Fulano.}
 \end{itemize}

lib/+mp/opt_model.m

@@ -536,7 +536,7 @@ function clear_cached_params(obj)
             %   TorF (logical): true or false, indicating whether any of the
             %       variables are binary or integer
 
-            TorF = ~mm.var.all_continuous();
+            TorF = ~mm.var.are_all_continuous();
         end
 
         function TorF = is_solved(mm)

lib/+mp/sm_variable.m

@@ -11,6 +11,7 @@
 % By convention, ``var`` is the variable name used for mp.sm_variable objects.
 %
 % mp.sm_variable Properties:
+%   * all_continuous - true if all variables are continuous, false otherwise
 %   * cache - struct for caching aggregated parameters for the set
 %
 % mp.sm_variable Methods:
@@ -21,6 +22,7 @@
 %   * display_soln - display solution values for variables
 %   * get_soln - fetch solution values for specific named/indexed subsets
 %   * parse_soln - parse solution for variables
+%   * are_all_continuous - return true if all variables are continuous, false otherwise
 %   * varsets_idx - return vector of indices into full :math:`\x` corresponding to ``vs``
 %   * varsets_len - return the total number of variables specified by ``vs``
 %   * varsets_x - return subset of :math:`\x` specified by ``vs``
@@ -29,14 +31,15 @@
 % See also mp.set_manager, mp.set_manager_opt_model.
 
 %   MP-Opt-Model
-%   Copyright (c) 2008-2024, Power Systems Engineering Research Center (PSERC)
+%   Copyright (c) 2008-2026, Power Systems Engineering Research Center (PSERC)
 %   by Ray Zimmerman, PSERC Cornell
 %
 %   This file is part of MP-Opt-Model.
 %   Covered by the 3-clause BSD License (see LICENSE file for details).
 %   See https://github.com/MATPOWER/mp-opt-model for more info.
 
     properties
+        all_continuous = [];
         % struct for caching aggregated parameters for variables
         cache = [];
     end     %% properties
@@ -173,6 +176,14 @@
                 obj.data.vu = subsasgn(obj.data.vu, sc, vu);    %% upper bound
                 obj.data.vt = subsasgn(obj.data.vt, sc, vt);    %% variable type
             end
+
+            %% update all_continuous
+            if isempty(obj.all_continuous)
+                obj.all_continuous = obj.are_all_continuous();
+            else
+                obj.all_continuous = obj.all_continuous && all(vt == 'C');
+            end
+
             if ~isempty(obj.cache)  %% clear cache of aggregated params
                 obj.clear_cached_params();
             end
@@ -214,7 +225,7 @@
             %
             % See also add, set_params.
 
-            if nargout > 3 || ~obj.all_continuous()
+            if nargout > 3 || ~obj.are_all_continuous()
                 have_vt = 1;
             else
                 have_vt = 0;
@@ -472,6 +483,15 @@
             if is_all && dN
                 obj.set_params_update_dims(dN, name, idx);
             end
+
+            %% update all_continuous
+            if u.vt
+                if isempty(obj.all_continuous)
+                    obj.all_continuous = obj.are_all_continuous();
+                else
+                    obj.all_continuous = obj.all_continuous && all(p.vt == 'C');
+                end
+            end
         end
 
         function obj = display_soln(obj, soln, varargin)
@@ -702,34 +722,39 @@
             end
         end
 
-        function TorF = all_continuous(obj)
-            % Return true of all variables are continuous, false otherwise.
+        function TorF = are_all_continuous(obj)
+            % Return true if all variables are continuous, false otherwise.
             % ::
             %
-            %   TorF = var.all_continous();
+            %   TorF = var.are_all_continuous();
             %
             % Output:
             %   TorF (logical) : true if all variables are continuous,
             %       false otherwise.
 
-            TorF = true;
-            if obj.get_N()
-                for k = 1:length(obj.order)
-                    t = obj.data.vt.(obj.order(k).name);
-                    if iscell(t)
-                        for j = 1:length(t(:))
-                            if any(t{j} ~= 'C')
+            if isempty(obj.all_continuous)
+                %% evaluate and cache value
+                TorF = true;
+                if obj.get_N()
+                    for k = 1:length(obj.order)
+                        t = obj.data.vt.(obj.order(k).name);
+                        if iscell(t)
+                            for j = 1:length(t(:))
+                                if any(t{j} ~= 'C')
+                                    TorF = false;
+                                    break;
+                                end
+                            end
+                        else
+                            if any(t ~= 'C')
                                 TorF = false;
                                 break;
                             end
                         end
-                    else
-                        if any(t ~= 'C')
-                            TorF = false;
-                            break;
-                        end
                     end
                 end
+            else    %% use cached value in all_continous property
+                TorF = obj.all_continuous;
             end
         end
 

lib/mpomver.m

@@ -26,7 +26,7 @@
 v = struct( 'Name',     'MP-Opt-Model', ...
             'Version',  '5.1-dev', ...
             'Release',  '', ...
-            'Date',     '17-Apr-2026' );
+            'Date',     '20-Apr-2026' );
 if nargout > 0
     if nargin > 0
         rv = v;