Skip to content

Conversation

@cePuerto
Copy link

@cePuerto cePuerto commented Nov 20, 2025

Description

Fixes decoder memory consumption.

Reduced number of threads for onnx to one. Only one onnx session can be run at the same time on the browser. Added try-catch clausules to different promises in decoding routine.

Related Pull Requests

At lib-jitisi-meet: internxt/lib-jitsi-meet#89

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

How Has This Been Tested?

One hour call in 4 different browsers (Chrome, Safari, firefox, Brave). Decoder, encoder activated as needed. No memory leaks were detected. Current features in production didn't break or crahed during test.

Additional Notes

@github-actions
Copy link

🚀 Preview Link: https://1aca9799.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://5aaa658c.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://1c0013b0.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://e43c1e34.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://56702c3b.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://7f932740.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://b83ab542.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://2694bb38.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://5da3dfed.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://e36f7cae.meet-web.pages.dev

@cePuerto cePuerto marked this pull request as ready for review November 20, 2025 17:40
@github-actions
Copy link

🚀 Preview Link: https://386d301e.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://47988e2c.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://48d7a9c4.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://74788df1.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://d035e02a.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://94912fbf.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://cf3b3ffb.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://9dc043e4.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://b48e92b6.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://8508dc49.meet-web.pages.dev

…alled. Deletes containers src when switching between streams
@github-actions
Copy link

🚀 Preview Link: https://ffd756f2.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://6f2e0bb9.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://474fc25d.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://5ba63ecf.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://ebc9c843.meet-web.pages.dev

@github-actions
Copy link

🚀 Preview Link: https://f283baa6.meet-web.pages.dev

Copy link
Member

@sg-gs sg-gs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As long as this only kicks with slow internet mode and does not crash the app, this works for me.

@cePuerto cePuerto merged commit 5ed2fd8 into main Dec 3, 2025
3 of 6 checks passed
@cePuerto cePuerto deleted the Decoder_memory_reduction branch December 3, 2025 11:12
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.

3 participants