Respect colorrange and support nonlinear interpolations #119
Conversation
| local_celldofs_field = reshape(@view(_celldofs[dof_range_]), (field_dim,length(cell.nodes))) | ||
| local_celldofs_field = reshape(@view(_celldofs[dof_range_]), (field_dim, length(dof_range_)÷field_dim)) |
There was a problem hiding this comment.
What exactly is fixed here? Both statements should be equivalent.
There was a problem hiding this comment.
Ah, bad commit message - makes it work for superparameteric elements (and subparametric).
There was a problem hiding this comment.
But I see now that this would fail below for subparametric and relies on that the linear dofs comes first for superparameteric (but I think they do...).
So probably a more complete solution would be desired...?
| @@ -33,7 +33,7 @@ keyword arguments are: | |||
| deformation_field=:default, | |||
| process=postprocess, | |||
| colormap=:cividis, | |||
| colorrange=(0,1), | |||
| colorrange=(0,0), | |||
There was a problem hiding this comment.
I think what you want is something like
do I understand correctly?
There was a problem hiding this comment.
That might work too - I don't know the internal details enough to say, just made the minimum possible intrusive change I could come up with.
Currently, the problem was just that the provided colorrange was just overwritten here
So any solution respecting the provided input would be good.
While working I discovered a few issues, and am using this as my "fixes branch", hence the unrelated changes.
Towards #103 as based on that branch
Fixes
solutionplotbeing respected (instead of being overwritten by the automatic range calculated from the data)