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
Disabling local video track causes crash #308
Comments
Can you tell me what version of Thanks, Ryan |
I used 2.3, yesterday I have updated to 2.4 and the crash still occurs.. Could it be something with renderers assigned to track? Or the room settings? |
Thanks for getting back so quick. I don't believe it has anything to do with room settings. I just tested this in our internal application and I am not seeing the crash. Is this crash occurring on the device where you are disabling the local video or on the remote participant side? Are you able to share the source showing how you are rendering the local video track? I would like to try and get a setup where I can reproduce the crash you are seeing locally. Thank you, Ryan |
@paynerc One thing that may be relevant is that even when you disable the local video track before connecting to room, it crashes only after the twilio connects to the room. Could it be the problem with room configuration, or with access token?
|
Having the same issue here, version 2.4.1. This is a collapsed log to see some timestamps, first row is our own log that indicates that the track has been disabled.
|
I am digging into this right now and will post as soon as I am able to replicate. Quick question, what iOS versions are you seeing this on? Ryan |
Both my devices are on iOS 12 GM |
Great! |
OK. That's good to know. I will test on the various devices I have and see if I can reproduce and if there is any iOS version relation. |
Also, what device models are you testing on? |
iPhone 8 Plus. |
OK. I have tested on both an iPhone 6Plus and an iPhone X with iOS 12 and am not seeing anything yet. I don't have an iPhone 8 Plus.. I have a 7 Plus with iOS 11 and was also not able to reproduce the issue. A few more questions for you on this:
I am interested to hear if the crash is related to the iPhone 8 Plus or happens on other devices as well. Ryan |
Stack trace (bt):
I also emailed the RoomSid to support@twilio.com (support ticket 1528481). |
See this on multiple devices |
Hey @aastlind, Thanks for the stack trace. I believe that, in 2.3.0 and later, we are making a bad assumption in the H.264 encoder about frame types, and formats. Disabling the Track causes software I420 buffers to be produced for the black frames, and our encoder was not expecting this type. Regards, |
@ceaglest: Only to confirm, I just run a test removing H.264 as preferred video codec (leaving VP9 and VP8 as remaining preferred codecs) and then it works fine to disable the track. Good to see that you know where to start! :) |
@aastlind I apologize for the issue. Yes, we have a bug in our H.264 video encoder. I am working on the fix and aiming to release it in a couple of days. Thanks. |
We released Twilio Video iOS Let us know if this has resolved your issue and then we'll close out this issue. Thanks for your patience while we diagnosed and fixed the issue. Ryan |
@paynerc |
HI @paynerc I am getting this crash again with 'TwilioVideo', '~> 5' When I try to disable localVideoTrack, app just freeze in debug mode and on TestFlight build it crashes. I have that crash log if you want. Also, I have sent an email of Sid Thank you |
Tagging @piyushtank on this issue as I am no longer with Twilio. Ryan |
I am using
TwilioVideo
for calls over the internet. I am trying to support disabling video.Based on the example code, I expected this would do the trick:
Where
localVideoTrack
is an instance ofTVILocalVideoTrack
created using:I used basically the same approach with the
TVILocalAudioTrack
to mute and unmute microphone, which works like a charm.However, with video, when I call:
I get a crash:
Any idea why this is happening?
The text was updated successfully, but these errors were encountered: