An error occurred while fetching the assigned milestone of the selected merge_request.
More staggered grid improvements
- Fix access to directions with mixed signs (
NW
,(-1/2, 1/2)
and the like). They were previously mapped to the wrong cell. - When storing fluxes on a staggered grid, the usual sign convention is that
fluxes point outward from the cell. Previously, we did not consistently respect that
(
staggered_access("E")
would return the same thing asstaggered_access("W")
did when called from the eastern-next cell). Now, when a field is declared asSTAGGERED_FLUX
, it returns an accessor with a prefactor of -1 in that case. The previous behavior where sign is not reversed is still useful when e.g. storing sums (e.g. mean values) instead of differenes (e.g. finite difference fluxes) on the staggered grid. -
staggered_vector_access
returns a vector/tensor full ofAccess
objects corresponding to the shape (index_shape[1:]
) of the staggered field
Edited by Michael Kuron
Merge request reports
Activity
Filter activity
added 1 commit
- a7460abb - staggered_access: optionally reverse sign for access via opposite direction
mentioned in merge request !93 (merged)
added 1 commit
- 2258e7dd - Field.staggered_vector_access and Field.center_vector for up to 3 index dimensions
mentioned in commit 8ca8b2ee
mentioned in merge request !109 (merged)
Please register or sign in to reply