Skip to content

Conversation

@bbayles
Copy link
Collaborator

@bbayles bbayles commented May 30, 2024

This PR adds dft and idft, contributed by @rhettinger in #843.

Additionally, the docs now have a separate section for the various math-related functions.

@bbayles bbayles linked an issue May 30, 2024 that may be closed by this pull request

class DiscreteFourierTransformTests(TestCase):
def test_basic(self):
xarr = [1, 2 - 1j, -1j, -1 + 2j]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest adding a comment that links back to the wikipedia example which is worked out step-by-step: # https://en.wikipedia.org/wiki/Discrete_Fourier_transform#Example.

@pochmann3
Copy link
Contributor

Title has a typo, says dft twice. And did you add doublestarmap here intentionally?

@bbayles bbayles changed the title Add dft and dft Add dft and idft May 31, 2024
@bbayles
Copy link
Collaborator Author

bbayles commented May 31, 2024

Thanks, I fixed the title.

The doublestarmap PR conflicted with this one.

@rhettinger
Copy link
Contributor

LGTM :-)

@bbayles bbayles merged commit 6944752 into master Jun 2, 2024
@bbayles bbayles deleted the issue-843-dft branch June 2, 2024 00:19
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.

Proposal: Discrete Fourier Transform

4 participants