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

butterworth now corrects the cutoff frequency for the double pass filter. #28

Merged
merged 1 commit into from Mar 31, 2015

Conversation

Projects
None yet
1 participant
@moorepants
Owner

moorepants commented Mar 31, 2015

  • Included the correction factor for nth order Butterworth filter.
  • Adjusted the butterworth tests to accommodate the change. This change
    unfortunately required larger tolerances for the test. If this was actually
    correct, I would have thought it would have required smaller tolerances. I am
    not sure why that is the case.
butterworth now corrects the cutoff frequency for the double pass fil…
…ter.

- Included the correction factor for nth order Butterworth filter.
- Adjusted the butterworth tests to accommodate the change. This change
  unfortunately required larger tolerances for the test. If this was actually
  correct, I would have thought it would have required smaller tolerances. I am
  not sure why that is the case.

moorepants added a commit that referenced this pull request Mar 31, 2015

Merge pull request #28 from moorepants/butterworth-correction-factor
butterworth now corrects the cutoff frequency for the double pass filter.

@moorepants moorepants merged commit 564d431 into master Mar 31, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@moorepants moorepants deleted the butterworth-correction-factor branch Mar 31, 2015

@moorepants moorepants referenced this pull request Mar 31, 2015

Open

Speed up inverse dynamics (Python version) #128

3 of 5 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment