fix minor regressions introduced with !86
-
DataHandling.array_like
did not preserve staggeredness. - Printing a staggered
FieldAccess
sometimes threw errors because ofint
vs.numpy.int64
. - Accessing an invalid staggered neighbor did not produce a good error message.
- I had reversed the index convention so that it was inconsistent with the documentation of
create_staggered_kernel
.
Edited by Michael Kuron
Merge request reports
Activity
added 1 commit
- 10fe510f - fix minor regressions introduced with staggered field access
You probably want to use
sympy.sympify
instead ofint
. That will also work if you use symbols for the offsets.Edit: sympify does not work with
np.int
. But you maybe still want allow Sympy symbols that are not concrete numbers.Edited by Stephan SeitzThese numbers come from
direction_string_to_offset
, which always returns anumpy.array(..., dtype=numpy.int64)
. So there is no way to make a symbol appear here.Edited by Michael Kuronmentioned in commit a723120e
Please register or sign in to reply