You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@kemkartanya lets chat about this here - anyone else want to weigh in? i personally think it would be super cool to have this - we want to make sure that we are only adding things which would be useful to have in a beginner friendly library and we aren't adding features which might be unnecessary. part of the redesign process was to trim the excess - but I feel like I thought of some good reasons to have this but can't remember them now haha
this PR introduces a P5SoundUtils constant that I think would be a good place to put these methods.
@ogbabydiesal I think this casts another vote in favor a p5.Sound namespace, so that calling midiToFreq for example can be p5.Sound.midiToFreq() instead of P5SoundUtils.midiToFreq().
Would be curious to hear if @ksen0 or @davepagurek have any thoughts on this?
for reference
freqToMidi()
midiToFreq()