mitmedialab / prg-raise-playground

Boilerplate for playing with and deploying Scratch 3.0 modifications!
https://mitmedialab.github.io/prg-extension-boilerplate/main/
MIT License
19 stars 39 forks source link

Come up with a cleaner fix for displaying sheet music of notes out of range #103

Closed DolevArtzi closed 2 years ago

DolevArtzi commented 2 years ago

Currently, if a note has frequency x which doesn't fit in the range [min,max] where min and max represent the extremes for frequencies of notes that are normally displayable in the sheet music, we set x to min or max for the purpose of sheet music visualization, depending on which end of the extreme x is on. This is a good temporary fix, and this fix would likely go unnoticed by a lot of users, but it may be confusing to hear multiple notes that clearly sound different but are displayed as the same in the sheet music. One potential thing we could do to distinguish these notes is color them red, perhaps. For now, there are print statements here: https://github.com/mitmedialab/prg-extension-boilerplate/blob/311c955b6262b26a1802e4bc588f4c88cd76926e/packages/scratch-vm/src/extensions/scratch3_musiccreation/sheetmusic.js#L912 and a little below it that show when a note is being adjusted.

DolevArtzi commented 2 years ago

https://github.com/mitmedialab/prg-extension-boilerplate/pull/122