Core (LV::AudioConvert): Rewrite audio sample conversion by kaixiong · Pull Request #381 · Libvisual/libvisual · GitHub
Skip to content

Core (LV::AudioConvert): Rewrite audio sample conversion - #381

Draft
kaixiong wants to merge 1 commit into
masterfrom
rewrite-audio-convert
Draft

Core (LV::AudioConvert): Rewrite audio sample conversion#381
kaixiong wants to merge 1 commit into
masterfrom
rewrite-audio-convert

Conversation

@kaixiong

@kaixiong kaixiong commented Jan 18, 2025

Copy link
Copy Markdown
Member

Did extensive testing of the audio sample conversion and found a tonne of egregious numerical errors.

This PR fixes all of the known bugs. Where the conversion accuracy is sacrificed for performance, I have quantified the errors (with proofs!).

I don't know if they were introduced when I ported the original C macros to C++ generics, or during its refinements over time with modern C++-isms. If the bugs are present in the C original, I can backport the fixes over to 0.4.x.

More (gory) details to follow.

@kaixiong

Copy link
Copy Markdown
Member Author

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants