-
Notifications
You must be signed in to change notification settings - Fork 5
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
Challenge: Generate code for Joplin - Gladiolus Rag #78
Comments
I like this approach. It gives us concrete bugs to work on. The cross-staff notations are particularly interesting. |
@rvilarl, would you attach the MusicXML file somewhere in this issue please? |
The score is this one https://musescore.com/user/58480/scores/1749906 |
@jaredjj3 the score produces this fault at https://vexml.dev/ Gladiolus_Rag__Scott_Joplin__1907.zip 17:12:14.620 (1296px) Error: [RuntimeError] BadArguments: Too few notes for beam. at new sf (https://vexml.dev/assets/index-e7Ei05G8.js:60:246688) at Wo.render (https://vexml.dev/assets/index-e7Ei05G8.js:60:1213988) at https://vexml.dev/assets/index-e7Ei05G8.js:60:1224036 at Array.map () at Jh.render (https://vexml.dev/assets/index-e7Ei05G8.js:60:1224027) at Zs.render (https://vexml.dev/assets/index-e7Ei05G8.js:61:5105) at mr.render (https://vexml.dev/assets/index-e7Ei05G8.js:63:174) at https://vexml.dev/assets/index-e7Ei05G8.js:63:7625 at Dc (https://vexml.dev/assets/index-e7Ei05G8.js:40:24283) at Ss (https://vexml.dev/assets/index-e7Ei05G8.js:40:42406) |
Good catch, I think we can fix this by adding a check here and making the |
@jaredjj3 also starting on measure 10 the second staff is missing. |
There must be a bug with beaming. This will be a good score to thoroughly test the system. |
The good news: no crash reading the XML file.
The challenge: quite a few things to fix.
The text was updated successfully, but these errors were encountered: