commaai
UI doesn't show camera feed for CTF replay route
Describe the bug
For the route in CTF:
# start the route reply
cd tools/replay
./replay '0c7f0c7f0c7f0c7f|2021-10-13--13-00-00' --dcam --ecam
# start the UI in another terminal
selfdrive/ui/ui
The UI does not switch to the onroad display. The UI is receiving data as the camera feed can be viewed in Settings -> Device -> Driver Camera, the issue is probably with the format of the CTF route itself.
Starting the demo replay (/replay --demo --dcam --ecam), stopping the demo, then starting the CTF replay allows the CTF replay to work.
OS Version
Ubuntu 20.04
openpilot version or commit
No response
Additional info
No response
This is an old route that uses the deprecated pandaState service for ignition, @adeebshihadeh should we deprecate the CTF or support the old service in UI?
I'd prefer regenerating the route. I think it should be easy: xx/projects/ctf.
I am running into this issue as well. I can also verify that the driver camera preview will work, but my UI does not switch to the onroad display, even after pausing/resuming the replay or even quitting and reloading the replay.
For benefit of others looking at this issue, Shane and Adeeb already triaged it above, and they're going to fix it by regenerating the CTF route itself. The old CTF route has the ignition-on state in a place we no longer look for it.
I assume we can checkout to 3 years ago. https://github.com/commaai/openpilot/commit/9f06d6efd869840863e89939bdc8e36816cb2b08
Nah its not the code, its the route that's stored on comma's server. Nag @sshane or @adeebshihadeh and they'll be able to regenerate that route and fix it. I hope they do
Ran into this today.
In the meantime, you can instead run selfdrive/debug/uiview.py as a workaround.
Right now does this block CTF participation? I am just getting started learning openpilot and this blog post recommends starting with the CTF that is having issues.
nah you can still do the CTF it just makes it a little trickier to get started
would love to see this updated, since it is in this state for over a year.
