Added LPF to delay module

[CuttyBang]

Added a LPF to the Delay module and chained it in. Creates dubby tape delay sound. Can be BPF if preferred. Freqs just need to be shaved on each pass. Thanks guys.

[zxqx]

Thanks for the PR! Will check this out later today.

[CuttyBang]

Might want to check my routing. Attempted to follow the patterns, but it looks like something is failing in testing....

[trivorak]

at the top of the delay module you need to import the filter class. Also where you're setting your defaults it shouldn't have ".node"

filter.setFilterType('lowpass'); filter.setFrequency(1000);

not
filter.node.setFilterType or filter.node.setFrequency

also where you're connecting your filter to your delay module, that should end with a semi-colon

[CuttyBang]

Aha! I knew there was something I wasn't doing right. I hadn't gotten the hang of this pattern yet. Sorry about that. I hope it still has some value!

Nathan McElwain

On Sep 22, 2016, at 4:41 PM, Drake Champagne notifications@github.com wrote:

at the top of the delay module you need to import the filter class. Also where you're setting your defaults it shouldn't have ".node"

filter.setFilterType('lowpass');
filter.setFrequency(1000);

not
'filter.node.setFilterType or filter.node.setFrequency'

also where you're connecting your filter to your delay module, that should end with a semi-colon

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

[zxqx]

@CuttyBang Hey no problem man. Do you want to update the PR and then I can merge it in?

[CuttyBang]

Absolutely. I'm gonna hop on it this afternoon, as soon as I can get back to my desk! Thank you for keeping the PR alive!

Nathan McElwain

On Sep 23, 2016, at 8:46 AM, Zak Angelle notifications@github.com wrote:

@CuttyBang Hey no problem man. Do you want to update the PR and then I can merge it in?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

[zxqx]

No problem! I've been busy but I'm back and working on getting this to a 1.0 state over the next few months. If you have any other ideas feel free to submit them.

Also, I made an update last night so that you can spin up an example synth locally in your browser if you wanna test any new features (see the Example section in the README file for setting it up).

[CuttyBang]

Repaired the paths and syntax error. Added a comment to explain the routing a bit.

[CuttyBang]

Wow, what a flake! Sorry man. I guess I just got completely sidetracked and spaced on this in the worst way possible. At any rate, I finally fixed it, and added a comment about it.

[zxqx]

Thanks man! Will check this out when I get home tomorrow. Appreciate the contribution.

[zxqx]

@CuttyBang Hey, just wanted to let you know we are reworking some of the library internals to simplify things. The Delay module shouldn't be changing too much (if at all), and I'll be spending some time porting it over later this week and looking at your code changes.

[zxqx]

@CuttyBang So wavedef has been restructured a bit - the Delay module is now located at src/modules/Delay.js. Otherwise, not much else has changed, save for stricter linting rules and a few additional methods here and there.

How do you feel about porting over this change to the new code base and adding unit tests in Delay.test.js?

[CuttyBang]

Yea man, sure. I'm game. I see all the restructuring going on. Looks great. I'll hop on this some time over the next few days.