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

Scale Degree is acting "wonky" #2411

Closed
pikurasa opened this issue Jul 16, 2020 · 7 comments · Fixed by #2428
Closed

Scale Degree is acting "wonky" #2411

pikurasa opened this issue Jul 16, 2020 · 7 comments · Fixed by #2428
Assignees
Labels

Comments

@pikurasa
Copy link
Collaborator

I am having trouble finding the pattern, but the scale degree block is not working well on its own.

2-spelling-systems-for-Scale-Degree.html.zip

As a snapshot, you can see that the first pitch is not C (even though it is in C major and starts from SD=1). It seems to be making this lower C its seventh scale degree. Also, the two voices are not matching in pitch.

Screenshot at 2020-07-16 11:06:17

@pikurasa
Copy link
Collaborator Author

Tested on Rawgit.

@aviral243
Copy link
Member

I'm probably subtracting 1 somewhere I shouldn't, or vice-versa. Must've happened when I refactored the function.
I'll take a look.

@pikurasa
Copy link
Collaborator Author

No worries!

Working on the docs right now!

@pikurasa
Copy link
Collaborator Author

I tested it and the sound works perfectly as expected for the test example.

However, when I try and export as Lilypond I get:

PitchBlocks.js:2041 Uncaught TypeError: Cannot read property 'indexOf' of undefined
    at ScaleDegree2Block.arg (PitchBlocks.js:2041)
    at Logo.parseArg (logo.js:841)
    at Logo.runFromBlockNow (logo.js:1691)
    at logo.js:1650
PitchBlocks.js:2041 Uncaught TypeError: Cannot read property 'indexOf' of undefined
    at ScaleDegree2Block.arg (PitchBlocks.js:2041)
    at Logo.parseArg (logo.js:841)
    at Logo.runFromBlockNow (logo.js:1691)
    at logo.js:1650

@walterbender
Copy link
Member

@pikurasa please test

@walterbender walterbender reopened this Jul 31, 2020
@walterbender
Copy link
Member

Screenshot from 2020-07-31 18-38-15

@pikurasa
Copy link
Collaborator Author

pikurasa commented Aug 7, 2020

Yes, this is working as expected.

I would like to add #2461 as I plan to reference it in documentation.

@pikurasa pikurasa closed this as completed Aug 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants