JFVM.jl icon indicating copy to clipboard operation
JFVM.jl copied to clipboard

"full" not defined in reshapeCell and reshapeInternalCell

Open SA8416 opened this issue 4 years ago • 1 comments
trafficstars

Attempts to use reshapeCell or reshapeInternalCell result in an error of:

ERROR: LoadError: UndefVarError: full not defined
Stacktrace:
 [1] reshapeCell(m::MeshStructure, phi::CellValue)
   @ JFVM D:\Programs\Julia-1.6.1\.julia\packages\JFVM\v1vqd\src\JFVMtools.jl:330

my temporary (and quite poor) workaround to this is:

Ti = internalCells(T)
        Ti = [Ti[1,1:end]';Ti;Ti[end,1:end]']
        Ti = [Ti[1:end,1] Ti Ti[1:end,end]]
        T = createCellVariable(m,Ti)

So that I may use the cellvariable T (which is a solution from solving a PDE) in functions that are dependant on T, but return errors when operating on the ghost values of T.

I will look into a better fix & branch it later this week.

Thanks.

SA8416 avatar Jul 11 '21 19:07 SA8416

Hi @SA8416 , Thanks for reporting this issue. One quick solution is to replace full, which is deprecated in Julia 1.0, with Matrix. Now it should be fixed with the new commit. Please let me know if it does not work.

simulkade avatar Jul 12 '21 08:07 simulkade