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

fix: fixes #522 with bound check before computing twiddles when domain has no precompute set #523

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

gbotrel
Copy link
Collaborator

@gbotrel gbotrel commented Jul 26, 2024

See #522 for description.

Copy link
Collaborator

@ivokub ivokub left a comment

Choose a reason for hiding this comment

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

LGTM

@ivokub ivokub linked an issue Sep 2, 2024 that may be closed by this pull request
@ivokub ivokub added the bug Something isn't working label Sep 2, 2024
@gbotrel gbotrel merged commit 20fab15 into master Sep 3, 2024
7 checks passed
@gbotrel gbotrel deleted the fix/issue_522 branch September 3, 2024 19:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: fft
2 participants