Skip to content

Commit

Permalink
Assembler is not optional in FEOperators
Browse files Browse the repository at this point in the history
  • Loading branch information
fverdugo committed Aug 29, 2019
1 parent 43f272f commit b1bf517
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/FESpaces/FEOperators.jl
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@ struct NonLinearFEOperator <:FEOperator
testfesp::FESpaceLike
trialfesp::FESpaceLike
assem::AssemblerLike
terms::Tuple{<:FETerm}
terms::NTuple{N,<:FETerm} where N
end

function NonLinearFEOperator(
Expand All @@ -278,7 +278,7 @@ function NonLinearFEOperator(
trialfesp::FESpaceLike,
terms::Vararg{<:FETerm})
assem = SparseMatrixAssembler(testfesp,trialfesp)
NonLinearFEOperator(testfesp,trialfesp,assem,terms...)
NonLinearFEOperator(testfesp,trialfesp,assem,terms)
end

function NonLinearFEOperator(
Expand Down
5 changes: 5 additions & 0 deletions test/FESpacesTests/VectorValuedNonLinearFEOperatorsTests.jl
Original file line number Diff line number Diff line change
Expand Up @@ -82,4 +82,9 @@ eh1 = sqrt(sum( integrate(h1(e),trian,quad) ))
@test el2 < 1.e-8
@test eh1 < 1.e-8


# Further tests
op = NonLinearFEOperator(V,U,t_Ω)
op = NonLinearFEOperator(V,U,t_Ω,t_Ω)

end # module
1 change: 1 addition & 0 deletions test/MultiFieldTests/MultiFEOperatorsTests.jl
Original file line number Diff line number Diff line change
Expand Up @@ -101,5 +101,6 @@ e2h1 = sqrt(sum( integrate(h1(e2),trian,quad) ))

op = LinearFEOperator(a,b,V,U,assem,trian,quad)
op = LinearFEOperator(V,U,t_Ω,t_Γ)
op = LinearFEOperator(V,U,t_Ω)

end

0 comments on commit b1bf517

Please sign in to comment.