Skip to content

Conversation

@FSAFTik
Copy link

@FSAFTik FSAFTik commented Oct 27, 2025

Purpose

Complete data collection application for snapping frames from the stream to hub under certain set conditions

Specification

None / not applicable

Dependencies & Potential Impact

Depends on snaps v2 from future depthAI 3.2.0 release and depthai-nodes branch : feat/dai_dev_req_TEST_GROUP

Deployment Plan

Requires both of the dependencies to be merged to main

Testing & Validation

Manual testing of all of the available snapping conditions in different situations

@FSAFTik FSAFTik requested a review from PetrNovota October 27, 2025 12:28
@klemen1999
Copy link
Collaborator

Couple of notes:

  • Do we need both Data Collection app and the Hub Snaps PoC? If I understand correctly these are two very similar apps where Hub Snaps is basically the updated of the "Data Collection" one? If that is the case I would only keep the better one and rename that one to "Data Collection" (or some better name but not the "hub snaps" as this is too obscure IMO)
  • IMO we should put performant POCs under the existing apps directory instead of creating a new one
  • We should update the test constants map to include new examples as they have some limitations on how can they be ran (i.e. the apps work only in standalone rvc4 I assume)

@FSAFTik FSAFTik added the testable PR is ready to be tested label Oct 30, 2025
@PetrNovota
Copy link

@FSAFTik is working on a refactor right now. This is not presentable in its current form

@PetrNovota
Copy link

@klemen1999

Do we need both Data Collection app and the Hub Snaps PoC?

where is this second app?

IMO we should put performant POCs under the existing apps directory instead of creating a new one

Agree, we will move all performant PoCs under apps

@FSAFTik
Copy link
Author

FSAFTik commented Oct 30, 2025

where is this second app?
I did delete it on current branch

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.

4 participants