Fix #421: unify arglina/arglinb/arglinc behavior by removing public m from PureJuMP wrappers

Author: arnavk23

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()
 

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()
 

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()