Biquad LR4 for module-lfe-lp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wednesday, April 03, 2013 08:03:25 AM Thomas Martitz wrote:
> Am 31.03.2013 05:56, schrieb Alexander E. Patrakov:
> > 2013/3/30 Justin Chudgar <justin at justinzane.com>:
> >>> The "unwanted frequencies are discarded" part is what you did wrong.
> >> 
> >> I disagree with this analysis for this use-case. If this implementation
> >> were in remix.c it would make sense. However, it works with fictitious,
> >> synthetic, duplicated, averaged, generally boooooooogus (can't find
> >> quite the right adjective) source data. This module does NOT do
> >> remixing. Though I would prefer this to be done in remix.c, many people
> >> have said that it should be in a module and I am simply trying to
> >> achieve what I want within the guidelines laid out by more experienced
> >> and smarter pulseaudio community members.> 
> > OK. Since there are contradicting requirements put on you by different
> > people, let there be two modules. One for your use case (discard high
> > frequencies from the LFE channel, add phase delays to other channels
> > accordingly), one for my use case (move bass to subwoofer from other
> > channels). And yes, our use cases both matter, and they differ also by
> > the selection of the source material. I was talking more with music
> > DVDs and BluRays in mind, you had CDs.
> > 
> > Still, I reserve the right to review the code.
> 
> Just don't expect him to implement your use case :)

Actually, for all the help he has given me, I'm going to give it a shot. 
Besides, assuming I understand him correctly, I think that it would be an 
excellent thing to eventually merge into remix.c replacing the simplistic 
averaging done now.

> _______________________________________________
> pulseaudio-discuss mailing list
> pulseaudio-discuss at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux