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
Simple Synth pitches are incorrect #204
Comments
@franky47 i would like to work on this |
@akshay4n, awesome, go ahead ! Ping me here if there's anything you need help with. |
@franky47 I took a look at #203 and went through https://www.inspiredacoustics.com/en/MIDI_note_numbers_and_center_frequencies and https://www.arduino.cc/reference/en/language/functions/advanced-io/tone/ The problem isn't quite clear to me |
I initially thought that the frequency values were off in the pitch list, but the problem is in reading the table. It starts at B0, which should map to MIDI note number 23. So the index at which we read the pitch should be |
See #203.
Correct values of pitches:
https://www.inspiredacoustics.com/en/MIDI_note_numbers_and_center_frequencies
Values must be rounded to the nearest integer, as the
tone
function does not accept floating point values.The text was updated successfully, but these errors were encountered: