Skip to content

feat(idtrackerai): report every missing video field at once, name loose exports, explain units - #154

Merged
mbel-tech merged 6 commits into
mainfrom
fatina-del-codice/idtrackerai-bare-export-followups
Oct 11, 2026
Merged

mbel-tech merged 6 commits into
mainfrom
fatina-del-codice/idtrackerai-bare-export-followups

Conversation

@mbel-tech

Copy link
Copy Markdown
Owner

Summary

Remaining items from the idtracker.ai real-dataset handoff (docs/tracker-formats/HANDOFF_idtrackerai_real_dataset_run.md), left over after #151 merged.

  • One error for all missing video fields. A bare export lacking frame rate and video size gets a single IDT_DICT_MISSING_KEY error naming every missing field and its option, in run and in validate. (A pickle without consent is still reported before the size: the file must be opened to know if it records one.)
  • scan names loose idtracker.ai exports (trajectories.h5/.npy/.parquet/.pickle, or a trajectories_csv folder, outside a trajectories/ folder) and says to put them in <name>/trajectories/. They are still not claimed as sessions.
  • Run README: a Units section under body-length calibration (why *_cm columns are empty), the reader options the user supplied (marked as not recorded in the files), and identity slots that are more than 90% missing.
  • IDT_VERSION_UNKNOWN is information, not a warning, for 4.x files.
  • Handoff and changelog updated.

Test plan

  • New tests in test_recognise.py, test_normaliser.py, test_readme.py, test_issue_batch.py
  • exporters, scan, readers, CLI, issue-batch: 1183 passed (5 entry-point tests deselected, they need an installed package)
  • tests/real_samples idtracker.ai CLI and sample tests: 22 passed
  • ruff clean
  • Known failures that also fail on a clean origin/main: native 3-D test_pixel_unchanged (2) and the serial/parallel native run test, plus the 3-D serial/parallel and fusion timing tests.

🤖 Generated with Claude Code

mbel-tech and others added 3 commits October 11, 2026 13:50
…se exports, explain units

- normaliser: fps, width and height missing together give one error naming all of them
- scan: names an idtracker.ai export lying outside a trajectories/ folder and says where to put it
- run README: Units note under body-length calibration, user-supplied reader options,
  identity slots that are more than 90% missing
- IDT_VERSION_UNKNOWN is information for 4.x files
- handoff and changelog updated

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…on, pin the FishTank survey, record which idtracker.ai rows Track2Data has run

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@mbel-tech
mbel-tech enabled auto-merge (squash) October 11, 2026 12:00
mbel-tech and others added 3 commits October 11, 2026 14:06
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
A bare idtracker.ai export added without its video size passed Validate with 'Ready to run.' and then failed in the run. The GUI action now adds Engine.session_read_issues(), as 'track2data validate' does.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…ts all-NaN columns with reasons

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@mbel-tech
mbel-tech merged commit 384f25c into main Oct 11, 2026
7 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