Skip to content

[New app] Add Live Video Search stack with Smart NVR and VSS Search integration#1539

Open
bhardwaj-nakul wants to merge 22 commits intoopen-edge-platform:mainfrom
bhardwaj-nakul:live-video-search
Open

[New app] Add Live Video Search stack with Smart NVR and VSS Search integration#1539
bhardwaj-nakul wants to merge 22 commits intoopen-edge-platform:mainfrom
bhardwaj-nakul:live-video-search

Conversation

@bhardwaj-nakul
Copy link
Contributor

Description

  • Introduced Docker Compose configurations for Smart NVR and VSS search mode.
  • Created user guide documentation including API reference, getting started, and system requirements.
  • Implemented setup script for environment configuration and container management.
  • Enhanced MQTT listener to use asynchronous connection for improved performance.

Fixes # (issue)

Any Newly Introduced Dependencies

No

How Has This Been Tested?

Tested in dev env

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes.
  • I have not introduced any 3rd party components incompatible with APACHE-2.0.
  • I have not included any company confidential information, trade secret, password or security token.
  • I have performed a self-review of my code.

- Introduced Docker Compose configurations for Smart NVR and VSS search mode.
- Created user guide documentation including API reference, getting started, and system requirements.
- Implemented setup script for environment configuration and container management.
- Enhanced MQTT listener to use asynchronous connection for improved performance.
enabled: true
max_disappeared: 30
objects:
track: [bicycle, car, motorcycle, person, vehicle]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not for immediate action: Target to make this configurable.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

with the current flow we expect the user to edit this config file to change the object detection classes

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Point was on a dictionary that can be dynamically created based on search query. We can take it as enhancement request. Right now, if the user queries for "Monkey on the tree", the monkey and tree will not be covered in object detection.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can check if we can only enable recording and not detection or tracking in frigate as this is not needed as part of the usecase

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. We should not exercise object detection in Frigate. Instead, it should be in the OD in MME and Data Prep.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay. I'll check and comment out the OD blocks

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed the OD blocks from config files

@bharagha bharagha added the GenAI label Jan 22, 2026
enabled: true
max_disappeared: 30
objects:
track: [bicycle, car, motorcycle, person, vehicle]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Point was on a dictionary that can be dynamically created based on search query. We can take it as enhancement request. Right now, if the user queries for "Monkey on the tree", the monkey and tree will not be covered in object detection.

Copy link
Contributor

@yogeshmpandey yogeshmpandey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bhardwaj-nakul Please update the relevant READMEs with the correct image tags

…ngs_per_second.txt, adjust Docker Compose for collector signals, and modify cleanup script to remove collector signals volume
@bhardwaj-nakul
Copy link
Contributor Author

@bhardwaj-nakul Please update the relevant READMEs with the correct image tags

Done. changed to 1.3.4-rc1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants