-
Notifications
You must be signed in to change notification settings - Fork 443
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
Speaker View mode not working #9257
Comments
Hi @nickvergessen. For the tests I have made, I can confirm that I have not followed any user. Edit: I realize that the speaker was not being promoted when I was testing the recording server and all recorded videos were showing a single user. |
@nickvergessen. Is it possible to debug the reception of the "I'm speaking signal" to the desktop client. |
Some debugging I made today. I don't know if I'm doing right.
This message repeated many times during the call. The browser is Firefox 111.0.1 and the other client is Talk Client 15.0.1 for Android. |
Not sure talk-android supports this: nextcloud/talk-android#88 |
Yes, I tested from Windows 7 - Firefox 111 to Ubuntu 20.04 - Firefox 111. Tomorrow I'll make a call between 3 or 4 participants. |
Today I made a call with two more participants. We were 3 in the call. Is it possible that the browser client is not responding to the "speaking" and "stoppedSpeaking" ? EDIT: |
I have made another call with 3 participants. This time I used Brave browser v1.50.125. The other two used Firefox 112.0 In my tests I could see that when Talk is used with signaling server the speaker view doesn't show the speaker but the first participant entering the call. Both browsers, Firefox and Brave showed the same results. |
Yesterday I found out that using Janus multistream versions (1.x) makes Talk to not recognize the control messages coming through the data channel because it is expecting the fixed label "JanusDataChannel" and the multistream version can use various data channels so it labels the channels with the corresponding stream number. |
^ @fancycode I guess you are aware of this? |
Janus 1.x is pretty much untested from my side. There is an open issue at strukturag/nextcloud-spreed-signaling#280 about testing it, but previous reports were that it "is working as with 0.x versions", so I didn't give this much priority. Probably not much I can do anyway as the DataChannels are between the client and Janus with no way to change anything from the signaling server side. |
The label of the data channel in 1.x Janus versions is the number of the channel, given that there can be more than one. |
How to use GitHub
Steps to reproduce
Expected behaviour
The focused participant is supposed to change automatically to the speaker
Actual behaviour
The focus remains on the first participant that joined the call
Talk app
Talk app version: (see apps admin page:
/index.php/settings/apps
)16.0.2
Custom Signaling server configured: yes/no and version (see additional admin settings:
/index.php/index.php/settings/admin/talk#signaling_server
)Yes, 1.1.2
Custom TURN server configured: yes/no (see additional admin settings:
/index.php/settings/admin/talk#turn_server
)Yes
Custom STUN server configured: yes/no (see additional admin settings:
/index.php/settings/admin/talk#stun_server
)Yes
Browser
Microphone available: yes/no
Yes
Camera available: yes/no
Yes
Operating system: Windows/Ubuntu/...
Ubuntu 20.04
Browser name: Firefox/Chrome/...
Firefox
Browser version: 85/96/...
111.0.1
Browser log
Server configuration
Operating system: Ubuntu/RedHat/...
Ubuntu 22.04
Web server: Apache/Nginx
Apache2
Database: MySQL/Maria/SQLite/PostgreSQL
Postgre
PHP version: 8.0/8.1/8.2
8.1
Nextcloud Version: (see admin page)
26.0.0
List of activated apps:
Nextcloud configuration:
Server log (data/nextcloud.log)
The text was updated successfully, but these errors were encountered: