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

Adding code for square wave oscillator and sawtooth oscillator. #19

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

zackchase
Copy link

Added code for square wave oscillator, based on code for sin oscillator.

@zackchase zackchase changed the title Adding code for square wave oscillator Adding code for square wave oscillator and sawtooth oscillator. Jul 28, 2014
@ssfrr
Copy link
Owner

ssfrr commented Aug 3, 2014

Hey Zach, just to summarize our email thread, this PR is mostly waiting on some tests and renaming SawtoothOsc to SawOsc. Also, at some point the default Saw and Square oscillators will be bandlimited so the behavior in this PR will be available with a bandlimit=false option. I can handle that part as part of merging it in.

@zackchase
Copy link
Author

Hi Spencer, yes! I'll be on it soon. Just was traveling over the weekend in
Karnataka.

Z

On Sun, Aug 3, 2014 at 8:33 AM, Spencer Russell [email protected]
wrote:

Hey Zach, just to summarize our email thread, this PR is mostly waiting on
some tests and renaming SawtoothOsc to SawOsc. Also, at some point the
default Saw and Square oscillators will be bandlimited so the behavior in
this PR will be available with a bandlimit=false option. I can handle
that part as part of merging it in.


Reply to this email directly or view it on GitHub
#19 (comment).

@zackchase
Copy link
Author

Hi Spencer - I just pushed the changes on my fork to incorporate the change
in nomenclature and add tests of SquareOsc and SawOsc. I believe the only
part of the tests that required any change were replacing the names
("SinOsc" with "SquareOsc" etc) and functions sin(phase) with the
appropriate functions.

On Tue, Aug 5, 2014 at 11:57 AM, zack [email protected] wrote:

Hi Spencer, yes! I'll be on it soon. Just was traveling over the weekend
in Karnataka.

Z

On Sun, Aug 3, 2014 at 8:33 AM, Spencer Russell [email protected]
wrote:

Hey Zach, just to summarize our email thread, this PR is mostly waiting
on some tests and renaming SawtoothOsc to SawOsc. Also, at some point the
default Saw and Square oscillators will be bandlimited so the behavior in
this PR will be available with a bandlimit=false option. I can handle
that part as part of merging it in.


Reply to this email directly or view it on GitHub
#19 (comment).

@ssfrr
Copy link
Owner

ssfrr commented Aug 19, 2014

Hey Zach,

Thanks for adding the tests. Do they pass on your system? On mine I'm getting a couple failures with the SquareOsc and then it throws an error.

@ssfrr
Copy link
Owner

ssfrr commented Aug 28, 2014

Little update - There were some issues with my travis config that were causing the travis tests to fail with all PRs. If you rebase on the latest master and re-push then travis should make it past the initial installation issues and then we'll be looking at the same test results.

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

Successfully merging this pull request may close these issues.

2 participants