Modifications to `add!`

JuliaFEM / FEMBase.jl

JuliaFEM base package (core functionality)

http://juliafem.org/

MIT License

16 stars 9 forks source link

Modifications to `add!` #41

Closed ahojukka5 closed 5 years ago

ahojukka5 commented 5 years ago

With these changes it's possible to give dofs1 and dofs2 eg. in sub-arrays or tuples, not only vectors. Also, data can be a long vector in row-major order (data_vec = vec(data)).

KristofferC commented 5 years ago

Makes sense to me, perhaps an @assert length(data) = length(dofs1) * length(dofs2) would be good to catch potential buggy input?

ahojukka5 commented 5 years ago

Makes sense to me, perhaps an @assert length(data) = length(dofs1) * length(dofs2) would be good to catch potential buggy input?

What a great idea! Already found a bug from test after adding an assertion. :)