Jishnub aqua (#546)

* Test using Aqua v0.8 * immutable path for zero * remove comments --------- Co-authored-by: Jishnu Bhattacharya <[email protected]>
JuliaMath · Dec 3, 2023 · dcfedd8 · dcfedd8
1 parent 0d1706e
commit dcfedd8
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 14 deletions.
diff --git a/Project.toml b/Project.toml
@@ -24,7 +24,7 @@ PolynomialsMakieCoreExt = "MakieCore"
 PolynomialsMutableArithmeticsExt = "MutableArithmetics"
 
 [compat]
-Aqua = "0.6, 0.7"
+Aqua = "0.8"
 ChainRulesCore = "1"
 ChainRulesTestUtils = "1"
 DualNumbers = "0.6"

diff --git a/src/polynomial-container-types/immutable-dense-polynomial.jl b/src/polynomial-container-types/immutable-dense-polynomial.jl
@@ -12,23 +12,24 @@ Immutable is a bit of a misnomer, as using the `@set!` macro from `Setfield.jl`
 """
 struct ImmutableDensePolynomial{B,T,X,N} <: AbstractDenseUnivariatePolynomial{B,T,X}
  coeffs::NTuple{N,T}
- function ImmutableDensePolynomial{B,T,X,N}(cs::NTuple{N}) where {B,N,T,X}
+ function ImmutableDensePolynomial{B,T,X,N}(cs::Tuple{S,Vararg{S}}) where {B,N,T,X, S}
+ m = length(cs)
+ m > N && throw(ArgumentError("Tuple too large for N"))
+ m < N && (cs = ntuple(i -> i <= m ? cs[i] : zero(T), Val(N)))
+ new{B,T,Symbol(X),N}(T.(cs))
+ end
+ function ImmutableDensePolynomial{B,T,X,N}(cs::Tuple{}) where {B,N,T,X}
  new{B,T,Symbol(X),N}(cs)
  end
 end
 
+
 ImmutableDensePolynomial{B,T,X,N}(check::Type{Val{false}}, cs::NTuple{N,T}) where {B,N,T,X} =
  ImmutableDensePolynomial{B,T,X}(cs)
 
 ImmutableDensePolynomial{B,T,X,N}(check::Type{Val{true}}, cs::NTuple{N,T}) where {B,N, T,X} =
  ImmutableDensePolynomial{B,T,X,N}(cs)
 
-# tuple with mismatched size
-function ImmutableDensePolynomial{B,T,X,N}(xs::NTuple{M,S}) where {B,T,S,X,N,M}
- p = ImmutableDensePolynomial{B,S,X,M}(xs)
- convert(ImmutableDensePolynomial{B,T,X,N}, ImmutableDensePolynomial{B,T,X,M}(xs))
-end
-
 # vector case with N
 function ImmutableDensePolynomial{B,T,X,N}(xs::AbstractVector{S}) where {B,T,S,X,N}
  ImmutableDensePolynomial{B,T,X,N}(ntuple(Base.Fix1(getindex, xs), Val(N)))
@@ -46,10 +47,13 @@ function ImmutableDensePolynomial{B,T,X,N}(c::S) where {B,T,X,N,S<:Scalar}
  cs = ntuple(i -> i == 1 ? T(c) : zero(T), Val(N))
  return ImmutableDensePolynomial{B,T,X,N}(cs)
 end
-ImmutableDensePolynomial{B,T,X}(::Val{false}, xs::NTuple{N,S}) where {B,T,S,X,N} = ImmutableDensePolynomial{B,T,X,N}(convert(NTuple{N,T}, xs))
+function ImmutableDensePolynomial{B,T,X}(::Val{false}, xs::Tuple{S,Vararg{S}}) where {B,T,S,X}
+ N = length(xs)
+ ImmutableDensePolynomial{B,T,X,N}(convert(NTuple{N,T}, xs))
+end
 ImmutableDensePolynomial{B,T,X}(xs::NTuple{N}) where {B,T,X,N} = ImmutableDensePolynomial{B,T,X,N}(convert(NTuple{N,T}, xs))
 ImmutableDensePolynomial{B,T}(xs::NTuple{N}, var::SymbolLike=Var(:x)) where {B,T,N} = ImmutableDensePolynomial{B,T,Symbol(var),N}(xs)
-ImmutableDensePolynomial{B}(xs::NTuple{N,T}, var::SymbolLike=Var(:x)) where {B,T,N} = ImmutableDensePolynomial{B,T,Symbol(var),N}(xs)
+ImmutableDensePolynomial{B}(xs::Tuple{T,Vararg{T}}, var::SymbolLike=Var(:x)) where {B,T} = ImmutableDensePolynomial{B,T,Symbol(var),length(xs)}(xs)
 
 # abstract vector. Must eat order
 ImmutableDensePolynomial{B,T,X}(::Val{false}, xs::AbstractVector, order::Int=0) where {B,T,X} =

diff --git a/test/aqua.jl b/test/aqua.jl
@@ -1,6 +1,3 @@
 using Aqua
 
-Aqua.test_all(Polynomials;
- unbound_args = false,
- stale_deps = false
- )
+Aqua.test_all(Polynomials)