watch3 arguments support

[andiradulescu]

This is a proposal to supporting arguments for watch3. This is what watch3 -h says with this PR:

Usage: selfdrive/ui/watch3 [options]

Options:
  -h, --help      Displays help on commandline options.
  --help-all      Displays help including Qt specific options.
  --cams <0,1,2>  cameras to decode, separated by a comma
  --nav           decode nav
  --qcam          decode qcamera
  --ecam          decode wide road camera
  --dcam          decode driver camera

Not using any argument, works as before.

[github-actions]

Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:

• Convert your PR to a draft unless it's ready to review
• Read the contributing docs
• Before marking as "ready for review", ensure:
  • the goal is clearly stated in the description
  • all the tests are passing
  • the change is something we merge
  • include a route or your device' dongle ID if relevant

[adeebshihadeh]

let’s just stick to the individual flags

[andiradulescu]

sure. also added --cams since that's how it's done in compressed_vipc.py.

should I also post another PR changing compressed_vipc.py to individual flags?

[adeebshihadeh]

sure

[deanlee]

I'm not quite clear on the purpose of this PR. Watch3 isn't standalone, it relies on Replay for video decoding and camera control. For example, using replay --demo -ecam --dcam displays three videos in Watch3, whereas replay --demo shows only one. Therefore, specifying decoding arguments in Watch3 doesn't actually determine which camera gets decoded; its primary function is to hide the unwanted camera views. automating this through VisionIpcClient::availableStreams would be more efficient.

[andiradulescu]

Totally correct: to be able to "show"/"hide" a specific camera. Or a set of two cameras, etc. Since showing just one camera in a 2x2, it's very small and with 3 empty "slots".

The idea to dynamically show the cameras based on VisionIpcClient::availableStreams is much nicer, indeed.

And yes, I naively copy-pasted the help descriptions:
--qcam decode qcamera
when I should've named them correctly:
--qcam show qcamera.

Thanks!

[github-actions]

This PR has had no activity for 14 days. It will be automatically closed in 3 days if there is no activity.

[github-actions]

This PR has been automatically closed due to inactivity. Feel free to re-open once activity resumes.