Skip to content

feat: add noise-suppression to voice messages#16465

Open
Antreesy wants to merge 4 commits intomainfrom
feat/3252/noise-cancellation--sapphi
Open

feat: add noise-suppression to voice messages#16465
Antreesy wants to merge 4 commits intomainfrom
feat/3252/noise-cancellation--sapphi

Conversation

@Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Dec 1, 2025

☑️ Resolves

🖌️ UI Checklist

🚧 Tasks

  • Adjust Rspack for imports
  • Make reusable module for sync processing
  • Connect voice messages

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Integrations with Files sidebar and other apps
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@Antreesy Antreesy added this to the 🍏 Next Major (33) milestone Dec 1, 2025
@Antreesy Antreesy self-assigned this Dec 1, 2025
@Antreesy Antreesy force-pushed the feat/3252/noise-cancellation--sapphi branch from 2711b13 to 746f08e Compare December 29, 2025 09:50
@Antreesy Antreesy marked this pull request as ready for review December 29, 2025 09:56
@Antreesy Antreesy requested a review from ShGKme December 29, 2025 09:56
@Antreesy Antreesy force-pushed the feat/3252/noise-cancellation--sapphi branch 2 times, most recently from ae746bf to 92f5581 Compare December 30, 2025 16:11
@Antreesy Antreesy marked this pull request as draft January 5, 2026 10:08
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the feat/3252/noise-cancellation--sapphi branch from 92f5581 to bf27cbf Compare February 20, 2026 12:45
@Antreesy Antreesy requested a review from DorraJaouad February 20, 2026 12:45
@Antreesy Antreesy marked this pull request as ready for review February 20, 2026 12:45
Copy link
Contributor

@DorraJaouad DorraJaouad left a comment

Choose a reason for hiding this comment

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

This works for voice messages as it is one audio input. Are you thinking about reusing it for calls?

@Antreesy
Copy link
Contributor Author

Are you thinking about reusing it for calls?

That's the idea. Mostly need to resolve the issue with device-id constraint

Copy link
Contributor

@DorraJaouad DorraJaouad left a comment

Choose a reason for hiding this comment

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

It works on voice messages, to be double-checked with calls edge cases

@Antreesy
Copy link
Contributor Author

Not merging it case noise supression in calls would need some major changes in util file

@DorraJaouad
Copy link
Contributor

noise supression in calls would need some major changes in util file

yeap, that's what I thought

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement feature: upload & shares & voice 📤🎙️ Sharing files into a chat and audio recordings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants