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

[Feature] Addition of support to mirroring of the video capture #11

Closed
wants to merge 1 commit into from

Conversation

kodypeterson
Copy link
Contributor

This adds a new feature to expose the ability to mirror the video capture stream. This does not change any default or existing behavior.

@ThibaultBee
Copy link
Member

Oh sorry, I missed your PR.

On iOS, the front camera is always mirrored. I thought this was your requirement.

In what use case, do you have to mirror or not the video?

@kodypeterson
Copy link
Contributor Author

@ThibaultBee Here is the write-up on explanation of the use-case

ThibaultBee/StreamPack#72 (comment)

@ThibaultBee
Copy link
Member

As far as I understand, you just need to mirror front cameras which is already the case in this project (not on Android yet).

So, the addition of an API to enable/disable mirroring does not seem usefull. Am I missing something? Do you need what is in the PR?

If you don't need this API, I will close the PR (not merge it).

@ThibaultBee
Copy link
Member

Rejected as we force the mirror for front camera now: see

videoCaptureUnit?.isVideoMirrored = camera.position == .front

@ThibaultBee ThibaultBee closed this Feb 5, 2024
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.

2 participants