Skip to content

Fix gaze estimation device timestamp propagation - #864

Merged
PetrNovota merged 1 commit into
mainfrom
codex/fix-gaze-estimation-device-timestamps
Sep 9, 2026
Merged

PetrNovota merged 1 commit into
mainfrom
codex/fix-gaze-estimation-device-timestamps

Conversation

@PetrNovota

Copy link
Copy Markdown
Contributor

Summary

  • propagate device timestamps through gaze-estimation crop configuration groups
  • preserve device timestamps on derived crop configs, head-pose tensors, and annotations
  • document the timestamp requirement for future changes

Problem

The three FrameCropper synchronization nodes received config groups with a default device timestamp of 0 ms while camera frames carried valid device timestamps. This prevented synchronization and caused continuous Sync warnings.

Testing

  • reproduced the zero-timestamp Sync warnings on an OAK4 before the fix
  • ran oakctl app run neural-networks/face-detection/gaze-estimation/ on an OAK4 after the fix; no Sync warnings appeared during the observation window
  • Python compileall
  • AGENTS.md validation
  • generated index check
  • Ruff, Ruff format, and mdformat pre-commit hooks

@PetrNovota PetrNovota added the testable PR is ready to be tested label Sep 8, 2026
@PetrNovota
PetrNovota merged commit 47df1fd into main Sep 9, 2026
14 of 15 checks passed
@PetrNovota
PetrNovota deleted the codex/fix-gaze-estimation-device-timestamps branch September 9, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testable PR is ready to be tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants