Skip to content

Fix scalar point-estimate shape for generic trackers - #5344

Merged
FlorianPfaff merged 2 commits into
mainfrom
agent/fix-scalar-point-estimates
Aug 18, 2026
Merged

Fix scalar point-estimate shape for generic trackers#5344
FlorianPfaff merged 2 commits into
mainfrom
agent/fix-scalar-point-estimates

Conversation

@FlorianPfaff

Copy link
Copy Markdown
Owner

What changed

  • Normalize AbstractFilter.get_point_estimate() with atleast_1d so a one-dimensional state is represented as a length-1 state vector instead of a scalar.
  • Add a regression that places a VonMisesFilter inside TrackManager and verifies the extracted point-estimate matrix has shape (1, 1).

Why

TrackManager.get_point_estimate() stacks per-track estimates along axis 1. Circular distributions can return scalar means, so a valid one-dimensional filter caused stacking to fail with an axis error. Other generic tracker code already assumes get_point_estimate().shape[0] exists, so normalizing at the base-filter interface makes the API consistent.

Validation

  • Added a targeted regression reproducing the scalar-filter case.
  • Diff is limited to the base point-estimate normalization and the regression test.

Created as a draft so repository CI can validate the backend matrix before merge.

@github-actions

Copy link
Copy Markdown
Contributor

MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Max errors Warnings Elapsed time
✅ COPYPASTE jscpd yes no no 5.72s
✅ JSON prettier 7 0 0 0 0.53s
✅ JSON v8r 7 0 0 3.87s
✅ MARKDOWN markdownlint 68 0 0 0 1.71s
✅ MARKDOWN markdown-table-formatter 68 0 0 0 0.35s
✅ PYTHON black 1983 10 0 0 55.16s
✅ PYTHON isort 1983 20 0 0 2.33s
✅ REPOSITORY betterleaks yes no no 1.19s
✅ REPOSITORY checkov yes no no 30.18s
✅ REPOSITORY git_diff yes no no 0.06s
✅ REPOSITORY secretlint yes no no 85.94s
✅ REPOSITORY syft yes no no 2.75s
✅ REPOSITORY trivy-sbom yes no no 1.92s
✅ YAML prettier 11 0 0 0 0.48s
✅ YAML v8r 11 0 0 13.66s
✅ YAML yamllint 11 0 0 0.79s

Notices

⚠️ Your configuration references items that have been removed from MegaLinter and are ignored: MAKEFILE_CHECKMAKE. See Removed linters to find their replacements.

See detailed reports in MegaLinter artifacts

Your project could benefit from a custom flavor, which would allow you to run only the linters you need, and thus improve runtime performances. (Skip this info by defining FLAVOR_SUGGESTIONS: false)

  • Documentation: Custom Flavors
  • Command: npx mega-linter-runner@10.0.0 --custom-flavor-setup --custom-flavor-linters PYTHON_BLACK,PYTHON_ISORT,COPYPASTE_JSCPD,JSON_V8R,JSON_PRETTIER,MARKDOWN_MARKDOWNLINT,MARKDOWN_MARKDOWN_TABLE_FORMATTER,REPOSITORY_CHECKOV,REPOSITORY_GIT_DIFF,REPOSITORY_BETTERLEAKS,REPOSITORY_SECRETLINT,REPOSITORY_SYFT,REPOSITORY_TRIVY_SBOM,YAML_PRETTIER,YAML_YAMLLINT,YAML_V8R

MegaLinter is provided by OX Security
Show us your support by starring ⭐ the repository

@FlorianPfaff
FlorianPfaff marked this pull request as ready for review August 18, 2026 19:16
@FlorianPfaff
FlorianPfaff merged commit 48318ba into main Aug 18, 2026
15 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant