Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ui: add raylib-based cameraview #33737

Closed
wants to merge 1 commit into from

Conversation

deanlee
Copy link
Contributor

@deanlee deanlee commented Oct 7, 2024

  1. Add Raylib-Based CameraView to selfdrive/ui/raylib
  2. Refactor watch3 to Use Raylib CameraView:
    Screenshot from 2024-10-07 23-02-02

Additional Note:

  1. The CameraView class is still incomplete, missing features like switching stream types, setting transformations, and C3X support. This PR is the first step in migrating to raylib and serves as a starting point for others who interested in contributing to the process.
  2. There is a longstanding bug in VisionIPC: vipcClient:connect returns true even when trying to connect to a non-existent stream type. A fix for this issue is here: commaai/msgq#624.

@github-actions github-actions bot added the ui label Oct 7, 2024
Copy link
Contributor

github-actions bot commented Oct 7, 2024

UI Preview

All Screenshots

@deanlee deanlee force-pushed the raylib_cameraview branch 5 times, most recently from d10f7c4 to e236892 Compare October 7, 2024 16:16
@deanlee deanlee marked this pull request as ready for review October 7, 2024 16:36
@deanlee deanlee marked this pull request as draft October 8, 2024 13:50
Copy link
Contributor

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

@github-actions github-actions bot added the stale label Oct 19, 2024
Copy link
Contributor

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

@github-actions github-actions bot closed this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant