Skip to content

Fix hardware accelerated rotation.#1450

Open
giorgi1324 wants to merge 1 commit intomrlt8:mainfrom
giorgi1324:fix_vaapi_and_qsv_rotations
Open

Fix hardware accelerated rotation.#1450
giorgi1324 wants to merge 1 commit intomrlt8:mainfrom
giorgi1324:fix_vaapi_and_qsv_rotations

Conversation

@giorgi1324
Copy link
Copy Markdown

Also fix another bug where rotation was always set to 'clock' due to reading raw camera names instead of capitalizing them before accessing env variables.

I tested this on a vaapi-enabled machine but looking at the following snippet from frigate ffmpeg code, I'm confident that this change also fixes things for qsv:
https://github.com/blakeblackshear/frigate/blob/6f9d9cd5a853b7c7cb4058ada8dccdca8cbc3f5b/frigate/ffmpeg_presets.py#L67-L68

Closes #1276

Also fix another bug where rotation was always set to 'clock'
due to reading raw camera names instead of capitalizing them
before accessing env variables.

I tested this on a vaapi-enabled machine but looking at the
following snippet from `frigate` ffmpeg code, I'm confident that
this change also fixes things for `qsv`:
https://github.com/blakeblackshear/frigate/blob/6f9d9cd5a853b7c7cb4058ada8dccdca8cbc3f5b/frigate/ffmpeg_presets.py#L67-L68
@IDisposable
Copy link
Copy Markdown

Since mrlt8 has been busy of late, I created a fork and merged the PR changes that have been pending for a while. Not trying to steal thunder, but I needed a stable source for updates.

The repo does all the Docker Hub publishing and everything, so feel free to click the ADD ADD-ON REPOSITORY button in the repo or grab the new docker image from here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Camera feed rotation with VAAPI acceleration is broken

2 participants