[Commits] [Sbozzolo/kuibit] 363b8d: kuibit: Support `sum` scalar reductions (#54)
Max Rizzo
noreply at github.com
Wed May 20 09:44:27 CDT 2026
Branch: refs/heads/master
Home: https://github.com/Sbozzolo/kuibit
Commit: 363b8d79acbca385a52c056128e42df7ef6fbed2
https://github.com/Sbozzolo/kuibit/commit/363b8d79acbca385a52c056128e42df7ef6fbed2
Author: Max Rizzo <maxrizzo99 at gmail.com>
Date: 2026-05-20 (Wed, 20 May 2026)
Changed paths:
M NEWS.md
M docs/cactus_scalars.rst
M kuibit/cactus_scalars.py
M tests/test_cactus_scalars.py
M tests/tov/output-0000/static_tov/admbase-curv.scalars.asc
Log Message:
-----------
kuibit: Support `sum` scalar reductions (#54)
Add support for the `sum` scalar reduction for Carpet and CarpetX outputs.
Also tighten the all-reductions header mapping introduced in #53. Unknown
header reduction names were previously registered in `_reduction_vars_columns`.
This usually had no practical effect because they were not exposed by
`ScalarsDir`, but ignoring them moves the code closer to a single source
of truth for supported reductions for Carpet/CarpetX respectively.
The testsuite seemed to be lacking a test that looks asks if every
supported reduction type is behaving for both Carpet/CarpetX. One test
is addded which loops over every kuibit reduction type, for both
Carpet and CarpetX, and checks that the kuibit read reductions match the
data from a direct numpy read.
To unsubscribe from these emails, change your notification settings at https://github.com/Sbozzolo/kuibit/settings/notifications
More information about the Commits
mailing list