Closed crlandsc closed 2 months ago
I updated the code with your suggestion for memory efficiency and cleaned it up a little more. I also added a stereo IR to verify compatibility with the unittest (which checks for both mono and stereo convolution). Everything checked out on my end, but let me know if there are other updates that should be made before merging.
Thanks! I will take a look at it tomorrow
Just merged your PR!
Great thanks so much!
I discovered that all IRs were only being loaded in as mono IRs. I updated the transform so that it can support both mono and multichannel IRs and channel-wise convolution to retain the correct spatial information.