Skip to content
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

Create allpass interpolation class #67

Closed
nwolek opened this issue Dec 26, 2015 · 4 comments
Closed

Create allpass interpolation class #67

nwolek opened this issue Dec 26, 2015 · 4 comments
Assignees

Comments

@nwolek
Copy link
Member

nwolek commented Dec 26, 2015

Detailed here. Foudn during research for issue #60.

To be implemented here:
https://github.com/jamoma/jamoma2/blob/master/include/core/JamomaInterpolate.h

Notes:

  • Uses feedback, therefore the interpolation would need to maintain a one sample history.
  • Efficiency is similar to linear interpolation.
  • Best when fractional value will be non-zero.
  • Not suitable for random access delays.
@nwolek nwolek self-assigned this Dec 26, 2015
@tap
Copy link
Member

tap commented Dec 27, 2015

This is interesting. I'm trying to figure out where we don't have random access. For example, a wavetable oscillator is a random-access situation, as is playing a soundfile. Maybe not the majority of the time but certainly in some cases.

What happens if you have a fixed delay and change the delay time? Is whatever happens "acceptable"?

Also interesting about this is that we could use an allpass building block inside of the implementation. Which I guess means we need to port this class from Jamoma1:
https://github.com/jamoma/JamomaCore/blob/master/DSP/extensions/FilterLib/includes/TTAllpass1a.h

@nwolek
Copy link
Member Author

nwolek commented Dec 28, 2015

@tap
Copy link
Member

tap commented Dec 28, 2015

The Jamoma::Allpass1 class could work in terms of creating a proof-of-concept. It would be much more efficient, however, to use a dedicated building-block with a fixed 1-sample delay.

In Jamoma1 that filter was called Allpass1a instead of Allpass1. There's got to be a better way of naming these though.

nwolek added a commit that referenced this issue Dec 31, 2015
nwolek added a commit that referenced this issue Dec 31, 2015
nwolek added a commit that referenced this issue Dec 31, 2015
nwolek added a commit that referenced this issue Dec 31, 2015
… passes test. leaving multiple comments about the limitations of this algorithm. see issue #67
@nwolek
Copy link
Member Author

nwolek commented Dec 31, 2015

Closed with this pull request
#76

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

No branches or pull requests

2 participants