Skip to content

Conversation

@jz592058
Copy link
Collaborator

@jz592058 jz592058 commented Nov 7, 2025

Adding a new example which stitches camera stream from multiple cameras and runs yolo model over the resulting panorama. It uses tiling to make it possible for the panorama to fit inside NN input resolution.

@klemen1999 klemen1999 added the testable PR is ready to be tested label Nov 7, 2025
Copy link
Contributor

@aljazkonec1 aljazkonec1 left a comment

Choose a reason for hiding this comment

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

Overall the logic looks good, I just left some small comments regarding the README. Lets just make sure the tests pass, otherwise thanks!

@jz592058
Copy link
Collaborator Author

@klemen1999 not sure why the test are failing now...

@klemen1999
Copy link
Collaborator

This is the error log from both runs (RVC2 and RVC4):
RuntimeError: Number of input streams for stitching must be at least 2.
This makes sense because the testing ATM doesn't support multi-device examples. In this case we should adjust the constants.py in tests/ with corrected reason for why its failing (test suite doesn't support multi-device testing) and set "platform": "all" so its skipped.

…se test suite does not support multi device testing
@jz592058
Copy link
Collaborator Author

@klemen1999 it seems that something else should be done since again we're seeing
RuntimeError: Number of input streams for stitching must be at least 2.

@klemen1999
Copy link
Collaborator

klemen1999 commented Nov 11, 2025

There is a typo in the path in the tests: you have stich instead of stitch
EDIT: Fixed

@klemen1999 klemen1999 merged commit c0213a6 into main Nov 11, 2025
8 checks passed
@klemen1999 klemen1999 deleted the nn-stitching branch November 11, 2025 12:31
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