[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