Skip to content

Harden movement metric aggregation and add export/API consistency tests#29

Merged
GuillermoDylan merged 2 commits into
mainfrom
fixes/general
May 7, 2026
Merged

Harden movement metric aggregation and add export/API consistency tests#29
GuillermoDylan merged 2 commits into
mainfrom
fixes/general

Conversation

@GuillermoDylan

Copy link
Copy Markdown
Contributor

What changed

  1. Public export surface hardening

    • Added a dedicated regression test suite to validate export consistency across:
      • pywib.__all__
      • pywib.core.__all__
      • pywib.utils.__all__
  2. Defensive movement metric behavior

    • Updated movement metric aggregators to behave deterministically when traces are incomplete:
      • velocity_metrics
      • acceleration_metrics
      • jerkiness_metrics
    • If df is provided and traces miss the required computed column, traces are recomputed.
    • Added/updated regression tests in movement test suite to cover:
      • recomputation path with incomplete traces + available df
      • expected error path with incomplete traces + df=None
      • precomputed-trace happy path

Validation

  • Movement and focused regression tests executed successfully.
  • Export-surface tests pass.
  • No lint issues in changed files.

@GuillermoDylan GuillermoDylan self-assigned this May 6, 2026
@GuillermoDylan GuillermoDylan added the enhancement New feature or request label May 6, 2026
@GuillermoDylan GuillermoDylan merged commit 29c32d6 into main May 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant