-
Notifications
You must be signed in to change notification settings - Fork 37
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
[Solution idea] Back camera cannot be streamed #69
Comments
There must be a regression as this was working before. Is the Selfie example app also not worming anymore? https://github.com/EricssonResearch/openwebrtc-examples/tree/master/ios/Selfie |
@stefanalund Selfie can only show the two cameras and not stream them via WebRTC, right? Are we talking about the same meaning of "streaming" here? 😄 Any thoughts on my question regarding the camera change during a running stream? |
@tobi512 yes, Selfie only shows the camera stream locally but it is using the same API's as for doing a remote WebRTC session. Unless there is a bug (regression) it should be possible to use the same API for setting the camera source also for remote sessions. |
Thanks for the quick answer! @stefanalund |
The local stream should™️ be the basis for the remote stream... |
@stefanalund Sorry for the late reply... On Chrome I get Are there any known errors at the moment? Couldn't verify my issue therefore... |
I have tested between ios nativedemo and chrome. Unluckily, i get the same error "OperationError: Error processing ICE candidate" in chrome after i press "call" in ios nativedemo. May i ask if there is any solution to tackle this problem? Thx |
Hey OWR-Team,
we're currently working with OWR in a productive project and therefore investigating on problems with the back camera.
On the current state, the back camera cannot be streamed, since
prepare_media_sessions_for_local_sources
has a booleanhave_video
that prevents the seconds video device from being prepared inprepare_media_session_for_source
.If we hardcode the index for the back camera into
prepare_media_sessions_for_local_sources
, so it's being prepared instead of the front camera (which is normally the first device), the video of the back camera is correctly sent to the remote.So, from my perspective, both cameras must be prepared (into 2 different media sessions?!) and the method
setVideoCaptureSourceByName
should also update the camera that is sent and not only the one thats shown locally.I'm willing to work on a solution for this, if I get support from your side (especially about the basic idea of the fix)!
@superdump @stefanalund Are you aware of the problem and do you agree on my solution idea?
PS: What steps are necessary for a camera change while the stream is running (new session description etc.?)
Thanks in advance!
The text was updated successfully, but these errors were encountered: