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
It is incorrectly handled and mapped somewhere; I couldn't pinpoint exactly where the inversion occurs.
The top 'magenta' note in the top-left of the grid is actually the MIDI note B4 at the end of the pitches array, however in the tile code it is indexed by y from the top-down on the screen and is index 0, therefore it gets mapped to the C of the notes array and gets assigned the color #BF4FA8 which is magenta:
I would expect the notes pitches and the colors which represent them to remain the same with any fix for this issue (i.e. there's no change to the user experience with a fix, no one would see or hear a change). But it'd be nice for the code to behave and represent the proper matching of note to color alignment for posterity. It's probably easiest to reverse the notes array, and then modify the color table to remap the colors based on what the actual notes used are vs. the incorrect mapping that exists today.
i.e. B should map to #BF4FA8 and A# should map to #8064C6, etc...
Thanks!
The text was updated successfully, but these errors were encountered:
If you look at this code here:
chrome-music-lab/melodymaker/app/data/Config.js
Lines 21 to 22 in e54bb41
And how it maps to colors here:
chrome-music-lab/melodymaker/app/data/Colors.js
Lines 31 to 48 in e54bb41
It is incorrectly handled and mapped somewhere; I couldn't pinpoint exactly where the inversion occurs.
The top 'magenta' note in the top-left of the grid is actually the MIDI note
B4
at the end of thepitches
array, however in the tile code it is indexed byy
from the top-down on the screen and is index0
, therefore it gets mapped to theC
of thenotes
array and gets assigned the color#BF4FA8
which is magenta:I would expect the notes pitches and the colors which represent them to remain the same with any fix for this issue (i.e. there's no change to the user experience with a fix, no one would see or hear a change). But it'd be nice for the code to behave and represent the proper matching of note to color alignment for posterity. It's probably easiest to reverse the
notes
array, and then modify the color table to remap the colors based on what the actual notes used are vs. the incorrect mapping that exists today.i.e.
B
should map to#BF4FA8
andA#
should map to#8064C6
, etc...Thanks!
The text was updated successfully, but these errors were encountered: