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.

Sign up for GitHub

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

Jump to bottom

IDFT should support onlyPositive array. #13

Closed
psambit9791 opened this issue Dec 30, 2020 · 1 comment
Closed

IDFT should support onlyPositive array. #13

psambit9791 opened this issue Dec 30, 2020 · 1 comment
Assignees
@psambit9791
Labels
enhancement New feature or request
Projects
Features List

Comments

@psambit9791
Copy link
Owner

IDFT right now supports only transforming array with same length as the signal. But a fully functional IDFT also needs to support transforming the half length sequences which are used as outputs from DFT of real-valued signals. This can be done because, for real valued signals, the second half is the mirrored complex conjugate of the first half.

This support needs to be added to the InverseDiscreteFourier class.
@psambit9791 psambit9791 added the enhancement New feature or request label Dec 30, 2020
@psambit9791 psambit9791 self-assigned this Dec 30, 2020
@psambit9791 psambit9791 added this to Issues ❌ in Features List Dec 30, 2020
@psambit9791
Copy link
Owner Author

Added enhancement in commit 1311e51.

@psambit9791 psambit9791 moved this from Issues ❌ to Fixed in Features List Dec 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@psambit9791 psambit9791

Labels
enhancement New feature or request
Projects
Features List
  
Fixed
Milestone
No milestone
Development

No branches or pull requests
1 participant
@psambit9791