Skip to content

Revert "Use livekit's Rust SDK instead of their swift SDK (#13343)"#20809

Merged
SomeoneToIgnore merged 1 commit intomainfrom
kb/back-to-swift
Nov 18, 2024
Merged

Revert "Use livekit's Rust SDK instead of their swift SDK (#13343)"#20809
SomeoneToIgnore merged 1 commit intomainfrom
kb/back-to-swift

Conversation

@SomeoneToIgnore
Copy link
Contributor

Issues found:

  • audio does not work well with various set-ups using USB
  • switching audio during initial join may leave the client with no audio at all
  • audio streaming is done on the main thread, beachballing certain set-ups
  • worse screenshare quality (seems that there's no dynamic scaling anymore, compared to the Swift SDK)

This reverts commit 1235d08.

Release Notes:

  • N/A

Issues found:

* audio does not work well with various set-ups using USB
* switching audio during initial join may leave the client with no audio at all
* audio streaming is done on the main thread, beachballing certain set-ups
* worse screenshare quality (seems that there's no dynamic scaling anymore, compared to the Swift SDK)

This reverts commit 1235d08.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Nov 18, 2024
@WeetHet
Copy link
Contributor

WeetHet commented Nov 18, 2024

Noooo, and just when I thought we would be able to package zed in nixpkgs properly 😭

@SomeoneToIgnore
Copy link
Contributor Author

I have a lot of noooo over this too, so I get the feeling.
We're back on track this month, hopefully.

@SomeoneToIgnore SomeoneToIgnore merged commit d92166f into main Nov 18, 2024
@SomeoneToIgnore SomeoneToIgnore deleted the kb/back-to-swift branch November 18, 2024 09:44
SomeoneToIgnore added a commit that referenced this pull request Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants