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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lost string when add tuning with # (sharp) #23

Open
sugizo opened this issue Sep 10, 2020 · 6 comments
Open

lost string when add tuning with # (sharp) #23

sugizo opened this issue Sep 10, 2020 · 6 comments

Comments

@sugizo
Copy link

sugizo commented Sep 10, 2020

api.ts

export const getTuningsForGuitar = () => ([
	{label: 'Regular', 'display': 'EBGDAE', strings: ['E4', 'B3', 'G3', 'D3', 'A2', 'E2'], tuningIdx: 0, name: 0},
	{label: 'Dropped D', 'display': 'EBGDAD', strings: ['E4', 'B3', 'G3', 'D3', 'A2', 'D2'], tuningIdx: 1, name: 1},
	{label: 'Double dropped D', display: 'DADGBD', strings: ['D4', 'A3', 'D3', 'G3', 'B2', 'D2'], tuningIdx: 2, name: 2},
	{label: 'Drop C', 'display': 'DAFCGC', strings: ['D4', 'A3', 'F3', 'C3', 'G2', 'C2'], tuningIdx: 3, name: 3},
	{label: 'Open G', 'display': 'DGDGBD', strings: ['D4', 'G3', 'D3', 'G3', 'B2', 'D2'], tuningIdx: 4, name: 4},
	{label: 'Major Third', 'display': 'ECG#ECG#', strings: ['E4', 'C4', 'G#3', 'E3', 'C3', 'G#2'], tuningIdx: 5, name: 5}
]);

lost strings

@walmik
Copy link
Collaborator

walmik commented Sep 11, 2020

this is a very interesting feature to add.. @sugizo ! Do you need CSS help for implementing it?

@sugizo
Copy link
Author

sugizo commented Sep 11, 2020

what i want to achieve is to learn the scale or arpeggio with another tuning, also posted another issues about this too, perhaps CSS issues or JS issues to generate fretboard and the Fret mark
nice to have feature that can support with more or less strings e.g. 4th strings bass, 5th strings bass, 7th strings guitar, etc

ref
https://en.wikipedia.org/wiki/List_of_guitar_tunings

thanks

@walmik
Copy link
Collaborator

walmik commented Sep 16, 2020

@sugizo I ve added support for this now, please take a look and let me know if it works as expected (I had to add flattened note names instead of sharps to make it work) https://scribbletune.github.io/johann/#/guitar

@sugizo
Copy link
Author

sugizo commented Sep 20, 2020

wonthefull thanks

@sugizo sugizo closed this as completed Sep 20, 2020
@sugizo
Copy link
Author

sugizo commented Sep 21, 2020

pardon, my mistake, tested on https://scribbletune.github.io/johann, just realize it define the note with Ab not G#
while on my local with your new fixed tested it with G#, so i reopen the issue again

./src/api.ts

{
   label: 'Major Third',
   display: 'ECG#ECG#',
   strings: ['E4', 'C4', 'G#3', 'E3', 'C3', 'G#2'],
   tuningIdx: 5,
   name: 5,
 },

result : 6 strings become 5 strings with lowest note is missing

screen_shot

result when click 'Flip fretboard vertically' : 6 strings become 4 strings

click flip fretboard vertically

thanks

@sugizo sugizo reopened this Sep 21, 2020
@sugizo
Copy link
Author

sugizo commented Nov 20, 2020

perhaps can learn from this
http://sugizo.pythonanywhere.com/music/fretboard/form_scale_in_fretboard
created using python and could share the code if you want

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