Enable CLI params in apps, tweak app setup and termination and disable UVC app with USB2 connections #618

VanDavv · 2022-01-24T11:09:39Z

in this PR:

Setup and terminate process is modified to correctly handle different OSes (tested on Windows and Mac)
Removed argcomplete dependency (as we're moving more into GUI and having it listed would require demo apps to have it as well)
Enabled same CLI arguments parsing for the apps as it is now for the demo
Disabled UVC app is connection speed is USB2 due to [BUG] UVC demo app does not work with OAK-D-Lite #616

apps/uvc/main.py

VanDavv added 6 commits January 24, 2022 10:46

Enable command line arguments parsing for apps

f741e1b

fix windows pathing issues

b36022b

Add pythonpath to app manager

47cb2cb

fix python path and app termination

9143ffb

disable USB2 connection in UVC app

0965db1

fix process wait logic in app manager

86d3a0f

VanDavv requested review from themarpe, Erol444 and SzabolcsGergely January 24, 2022 11:09

VanDavv mentioned this pull request Jan 24, 2022

[BUG] UVC demo app does not work with OAK-D-Lite #616

Open

SzabolcsGergely reviewed Jan 25, 2022

View reviewed changes

apps/uvc/main.py Outdated Show resolved Hide resolved

apps/uvc/main.py Outdated Show resolved Hide resolved

VanDavv added 2 commits January 25, 2022 20:20

Update exit codes

4f079aa

send output to sys.stderr

cb38edb

SzabolcsGergely approved these changes Jan 25, 2022

View reviewed changes

add ProcessLookupError handle

f15cf1b

VanDavv merged commit 2964432 into main Jan 25, 2022

VanDavv deleted the uvc_usb2 branch January 25, 2022 19:41

Provide feedback