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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typos in the Source Code #35

Closed
mishkov opened this issue Oct 25, 2021 · 7 comments
Closed

Typos in the Source Code #35

mishkov opened this issue Oct 25, 2021 · 7 comments

Comments

@mishkov
Copy link
Contributor

mishkov commented Oct 25, 2021

Hello there 馃憢

I think I found the typo in the rfft documentation:

/// - [x] A Array with the input

Please correct me if I am wrong. But if it is typo, can I create a pull request to fix it?

One more typo in the fft.dart comment:

// It is power of 2, apply cooley turkey algorithm

As I know there is no "cooley turkey algorithm" 馃檪

@polotto
Copy link
Collaborator

polotto commented Oct 25, 2021

You can do a PR to fix the typos, that will improve the documentation. About the Cooley Tukey, this is official name of FFT algorithm (https://en.wikipedia.org/wiki/Cooley%E2%80%93Tukey_FFT_algorithm). There are a plenty of them: DFT recursive, DFT divide and conquer, FFT Cooley Tukey, FFT for non power of 2 (Chirp-z / Bluestein's algorithm), etc.

@mishkov
Copy link
Contributor Author

mishkov commented Oct 25, 2021

I mean, is the letter "r" definitely in the name "Cooley Turkey"?

chrome_BSkRBI7Icx

It looks confusing.

@polotto
Copy link
Collaborator

polotto commented Oct 25, 2021

I got it. Yes, It's a mistake... anyway, go ahead do the PR fixing all the typos.

@mishkov
Copy link
Contributor Author

mishkov commented Oct 25, 2021

Ok.

@polotto
Copy link
Collaborator

polotto commented Nov 22, 2021

Hello there

Will you do a PR to fix these typos? I need to close this issue.

@mishkov mishkov mentioned this issue Jan 6, 2022
@mishkov
Copy link
Contributor Author

mishkov commented Jan 20, 2022

Fixed typos is merged

@mishkov mishkov closed this as completed Jan 20, 2022
@polotto
Copy link
Collaborator

polotto commented Jan 20, 2022

Update to 0.0.2-dev.7 to get your PR.

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

No branches or pull requests

2 participants