Skip to content

Smartstack Scaffolding - Stack Workers#440

Open
timbeccue wants to merge 1 commit intomainfrom
feature/smartstack-data-model
Open

Smartstack Scaffolding - Stack Workers#440
timbeccue wants to merge 1 commit intomainfrom
feature/smartstack-data-model

Conversation

@timbeccue
Copy link
Contributor

@timbeccue timbeccue commented Mar 8, 2026

Summary

This is one of two related PRs which have been split for easier reviews. In this PR, we first add the stacking core and data model. The next PR (441) will integrate these into the pipeline.

  • Add StackFrame DB model and helper functions to keep track of frames during smartstack observations
  • Add stacking worker (Check redis for new subframes, stack completion check, stack timeout detection)
  • Add stacking supervisor (runs a separate stack worker per camera)
  • Modify run_pipeline_stages: add return value with the reduced image filepath
  • unit tests

Overall, these changes are intended to get smartstacking ready with everything except the stacking algorithm itself.

One particular change I want to flag is that stage_utils.py, run_pipeline_stages used to have no return value, but now returns the images that were just processed, in order to easily pass the filepath to the stack db and worker. I don't think this causes problems in any existing calling code, but since it modifies a core banzai function I want to be sure.

@timbeccue timbeccue marked this pull request as ready for review March 9, 2026 15:25
@timbeccue timbeccue changed the title Add smart stacking data model, worker, and supervisor Smartstack Scaffolding - Stack Workers Mar 9, 2026
@timbeccue timbeccue requested a review from cmccully March 9, 2026 15:26
@timbeccue timbeccue force-pushed the feature/smartstack-data-model branch from 0ff1fc9 to 51a6fe0 Compare March 16, 2026 15:47
@timbeccue timbeccue force-pushed the feature/local-caching branch from aac53e8 to acaf457 Compare March 17, 2026 20:54
@timbeccue timbeccue force-pushed the feature/smartstack-data-model branch from 51a6fe0 to 8a3c7da Compare March 17, 2026 21:12
@timbeccue timbeccue changed the base branch from feature/local-caching to main March 18, 2026 21:34
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