-
Notifications
You must be signed in to change notification settings - Fork 109
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
Include waveform when sending voice messages #1650
Conversation
4687306
to
d767f1f
Compare
- New `AudioLevelCalculator` that outputs dB0v rescaled to the [0;1] range. - `VoiceRecorder` now stores the audio levels sampled while recording, then resamples them to 100 samples to use as waveform preview. - Waveform data is carried all the way as a `List<Float>` and converted to `List<Int>` in the [0;1024] range as per matrix spec only before sending it.
d767f1f
to
ef9e448
Compare
Kudos, SonarCloud Quality Gate passed! |
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #1650 +/- ##
===========================================
+ Coverage 63.01% 63.03% +0.02%
===========================================
Files 1204 1205 +1
Lines 31013 31039 +26
Branches 6389 6395 +6
===========================================
+ Hits 19543 19566 +23
Misses 8528 8528
- Partials 2942 2945 +3
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
AudioLevelCalculator
that outputs dB0v rescaled to the [0;1] range.VoiceRecorder
now stores the audio levels sampled while recording, then resamples them to 100 samples to use as waveform preview.List<Float>
and converted toList<Int>
in the [0;1024] range as per matrix spec only before sending it.