From 4dc16980f099eae00554b3eef3f123aef88bd79f Mon Sep 17 00:00:00 2001 From: arnavk23 Date: Thu, 4 Jun 2026 14:43:18 +0530 Subject: [PATCH] Fix #421: unify arglina/arglinb/arglinc behavior by removing public `m` from PureJuMP wrappers --- src/PureJuMP/arglina.jl | 5 ++--- src/PureJuMP/arglinb.jl | 5 ++--- src/PureJuMP/arglinc.jl | 5 ++--- 3 files changed, 6 insertions(+), 9 deletions(-) diff --git a/src/PureJuMP/arglina.jl b/src/PureJuMP/arglina.jl index 7828211e4..6a74958d2 100644 --- a/src/PureJuMP/arglina.jl +++ b/src/PureJuMP/arglina.jl @@ -12,9 +12,8 @@ export arglina "Linear function with `n` parameters and `m` observations - full rank" -function arglina(args...; n::Int = default_nvar, m::Int = 2n, kwargs...) - m < n && @warn("arglina: must have m ≥ n") - m = max(m, n) +function arglina(args...; n::Int = default_nvar, kwargs...) + m = 2 * n nlp = Model() diff --git a/src/PureJuMP/arglinb.jl b/src/PureJuMP/arglinb.jl index 7e30e0649..2b35d1c53 100644 --- a/src/PureJuMP/arglinb.jl +++ b/src/PureJuMP/arglinb.jl @@ -12,9 +12,8 @@ export arglinb "Linear function with `n` parameters and `m` observations - rank 1" -function arglinb(args...; n::Int = default_nvar, m::Int = 2n, kwargs...) - m < n && @warn("arglinb: must have m ≥ n") - m = max(m, n) +function arglinb(args...; n::Int = default_nvar, kwargs...) + m = 2 * n nlp = Model() diff --git a/src/PureJuMP/arglinc.jl b/src/PureJuMP/arglinc.jl index 4bdee4b9a..924f6997b 100644 --- a/src/PureJuMP/arglinc.jl +++ b/src/PureJuMP/arglinc.jl @@ -12,9 +12,8 @@ export arglinc "Linear function with `n` parameters and `m` observations - rank 1, zero columns and rows" -function arglinc(args...; n::Int = default_nvar, m::Int = 2n, kwargs...) - m < n && @warn("arglinc: must have m ≥ n") - m = max(m, n) +function arglinc(args...; n::Int = default_nvar, kwargs...) + m = 2 * n nlp = Model()