Skip to content

MOD: tracker version 2, FIX:OSM - #94

Merged
RBhupi merged 13 commits into
ARM-DOE:mainfrom
RBhupi:devel
Nov 5, 2026
Merged

RBhupi merged 13 commits into
ARM-DOE:mainfrom
RBhupi:devel

Conversation

@RBhupi

@RBhupi RBhupi commented Nov 5, 2026 •

Copy link
Copy Markdown
Collaborator

PR tags (required)

Select all that apply.

  • docs-only
  • tests-only
  • tiny-change (<5 lines, single class/file)
  • code-change
  • scientific/algorithmic
  • design/architecture
  • GUI

What changed

  • Tracker v2: footprints grown by a size-dependent radius, gates and no crossing links
    Latent tracks, splits/merges by overlap thresholds with identity by size-intensity score.
  • segmentation seeds carry across scans.

Why

Brings the main branch to the tracker version 2.

Issues closed

Closes #71

Target version (required)

Aiming for: next minor

Validation

Mark each item as done or N/A. Add one short evidence line for each item.

  • I ran ruff
  • I ran import-linter
  • I ran tests

AI usage & manual verification

If AI tools were used, complete this section.

  • AI used: yes
  • Model/tooling: Claude Code 🤖
  • Tasks delegated to AI: implementation assistance and tests under author direction; branch comparison and PR description
  • Manual verification performed: algorithm design and evaluation on real KHTX/KRLX data by the author; results reported in the JTech submission

Reviewer guide

Where should review start, and what are the key files/areas?

  • Entry point: src/adapt/modules/tracking/README.md (algorithm summary), then src/adapt/modules/tracking/module.py
  • Key files/areas (1-3): modules/tracking/matching/ (geometry, gates, assignment), modules/tracking/lineage.py + latent.py, contracts/decisions.py
  • Expected review time: ~2 h

Impact note (optional): Breaking config change — the tracker: keys are new

@codecov-commenter

codecov-commenter commented Nov 5, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.87613% with 88 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.36%. Comparing base (94a6039) to head (c3fb815).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/adapt/modules/tracking/module.py 90.43% 21 Missing and 14 partials ⚠️
src/adapt/persistence/track_store.py 76.92% 5 Missing and 4 partials ⚠️
src/adapt/modules/tracking/observations.py 95.33% 3 Missing and 4 partials ⚠️
src/adapt/persistence/decision_store.py 82.50% 4 Missing and 3 partials ⚠️
src/adapt/modules/acquisition/module.py 81.25% 4 Missing and 2 partials ⚠️
src/adapt/modules/detection/module.py 96.25% 2 Missing and 4 partials ⚠️
src/adapt/configuration/schemas/user.py 87.50% 1 Missing and 3 partials ⚠️
src/adapt/runtime/orchestrator.py 33.33% 3 Missing and 1 partial ⚠️
src/adapt/modules/tracking/events.py 92.00% 1 Missing and 1 partial ⚠️
src/adapt/modules/tracking/latent.py 93.75% 1 Missing and 1 partial ⚠️
... and 5 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
+ Coverage   89.62%   90.36%   +0.73%     
==========================================
  Files         113      116       +3     
  Lines        6575     7348     +773     
  Branches      827      918      +91     
==========================================
+ Hits         5893     6640     +747     
- Misses        476      483       +7     
- Partials      206      225      +19     
Flag Coverage Δ
unittests 90.31% <93.87%> (+0.74%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@RBhupi
RBhupi merged commit 7f92a70 into ARM-DOE:main Nov 5, 2026
12 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.

OpenStreetMap Access blocked: App is not following the tile usage policy

2 participants