depthai icon indicating copy to clipboard operation
depthai copied to clipboard

[Feature-Request] Support Mono Global Shutter cameras in all three ports

Open asa opened this issue 4 years ago • 11 comments

Start with the why:

It would be beneficial to have the camera interfaces be modular so that a mono camera could be used in all three ports of the system, or that RGB cameras could be used in all three ports for instance.

Move to the what:

Support Mono cameras in any port, with the ability to run all three pipelines fully, trigger frames syncronously etc.

Move to the how:

on Discord it was mentioned there needed to be some abstraction of the camera driver code to support this feature to remove the hardcoded notion of Left, Right and RGB for the camera ports.

asa avatar Aug 12 '21 00:08 asa

Thanks yes we agree this will be super useful and we are brainstorming and discussing the underlying implementation that would be required now.

Luxonis-Brandon avatar Aug 12 '21 03:08 Luxonis-Brandon

+1 for RGB cameras on all ports, AR0234 for example.

jlecoeur avatar Aug 25 '21 18:08 jlecoeur

Thanks @jlecoeur and @asa so as a heads up if this is needed now we can prepare branches to support it. Just let us know.

We're working on the underlying architecture to make this just plug and play. But for now we can just make a branch that expects 3x AR0234 for example.

Luxonis-Brandon avatar Aug 25 '21 22:08 Luxonis-Brandon

Yes please!!! I would use it immediately.

asa avatar Aug 25 '21 22:08 asa

Thanks. Will prepare. If you're not already in our Discord, please join and we can share the temporary branch there. @alex-luxonis is likely who will make it.

Luxonis-Brandon avatar Aug 25 '21 23:08 Luxonis-Brandon

Yes, am on discord as @asa

On Wed, Aug 25, 2021 at 4:25 PM Luxonis-Brandon @.***> wrote:

Thanks. Will prepare. If you're not already in our Discord https://discord.gg/EPsZHkg9Nx, please join and we can share the temporary branch there. @alex-luxonis https://github.com/alex-luxonis is likely who will make it.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/luxonis/depthai/issues/449#issuecomment-905941538, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAXPGP66E7WNT7FPH2GOXLT6V3XNANCNFSM5B73FYMQ .

asa avatar Aug 26 '21 00:08 asa

Great!

jlecoeur avatar Aug 26 '21 04:08 jlecoeur

Thanks. Will prepare. If you're not already in our Discord, please join and we can share the temporary branch there. @alex-luxonis is likely who will make it.

any updates on getting a branch out for this?

asa avatar Sep 09 '21 18:09 asa

Yes actually it just now went live: https://github.com/luxonis/depthai-python/pull/365

Good timing!

Luxonis-Brandon avatar Sep 09 '21 18:09 Luxonis-Brandon

@asa You could try installing the depthai from the above PR / branch multi_cam_support: python3 examples/install_requirements.py

And then to run for example 3x OV9282: python3 tests/cam_test.py -cams rgb,m left,m right,m

alex-luxonis avatar Sep 09 '21 18:09 alex-luxonis

Cosmic timing! Yes, this works great. You all are amazing!

asa avatar Sep 09 '21 19:09 asa