Error

[bflynn882]

Hello, I'm using Windows Subsystem for Linux and got an error after uploading my musicxml file. Attached are two screenshots of the error. Any help would be appreciated, looking forward to using this. Thanks!

Bryan

[NeroBlackstone]

Hi, thanks for your feedback, please upload your .musicxml file in this issue.
Or send your musicxml to [email protected] if you don't want to public sheet music.

[NeroBlackstone]

I have downloaded this musicxml, maybe the problem is "repeat notation", I never consider this situation.

[bflynn882]

Hello, so it still gave me an error after using another song with no repeats. It got farther this time though, I feel like it was almost complete. It took about an hour to do before it gave me the error, probably because my pc is crap. Attached is the error screenshot and the new xml song. Thanks for your help.

…

On Mon, May 8, 2023 at 12:00 AM Nero Blackstone​ ***@***.***> wrote: I have downloaded this musicxml, maybe the problem is "repeat notation", I never consider this situation. — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALTJEWVPXGHX2N4ZXFTB6C3XFCKZDANCNFSM6AAAAAAXZLQDJE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[NeroBlackstone]

Thanks for your feedback. Please send your piano sheet to my email. I will check it when I have time.

[NeroBlackstone]

What is your computer hardware configuration？Program should finished in few minutes.

[bflynn882]

I sent my sheet music xml file in my previous email. I'll try to do it on another computer and see if it works. Would it work on windows 10? Or does it have to be Linux? Like I said before, I have windows subsystem for Linux on the PC I tried this program with. Just wondering if it would work on just windows 10. Thanks.

…

On Tue, May 9, 2023, 5:45 AM Nero Blackstone​ ***@***.***> wrote: Thanks for your feedback. Please send your piano sheet to my email. I will check it when I have time. — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALTJEWXVMGYZW4DYONHNKJ3XFIG2LANCNFSM6AAAAAAXZLQDJE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[NeroBlackstone]

You can try it, I can't guarantee it will work correctly, I tested this package only on Linux.

[bflynn882]

So I got it to work with a random piece that I downloaded. It was River Flows in You. I'll attach the song in an email I'll send you right now because it won't let me upload a musicxml file on github. But it still won't work with the song Champagne Problems without any repeats. I'm still getting the bounds error. I'd appreciate it if you could help whenever you get a chance. Thanks.

[bflynn882]

Here's the musicxml file that worked (river flows in you). Also attached is the song Champagne Problems that gives the bounds error. I don't know why it won't work. There are no repeats or anything. Thanks again for your help.

…

On Tue, May 9, 2023 at 8:50 PM Nero Blackstone​ ***@***.***> wrote: You can try it, I can't guarantee it will work correctly, I tested this package only on Linux. — Reply to this email directly, view it on GitHub <#1 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALTJEWVREQKRX7TVVYUBRMDXFLQ5LANCNFSM6AAAAAAXZLQDJE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[NeroBlackstone]

I'll attach the song in an email I'll send you right now because it won't let me upload a musicxml file on github.

You could compress musicxml to zip file and upload.

[NeroBlackstone]

I have confirmed a bug from music21, some bounds errors can't be fixed by myself.

[bflynn882]

Ok, thanks a lot.

[michael-nhat]

There seem tobe many bugs, I export from midi file download from musescore org to musicxml (by musescore4, on archlinux), and files meet diffirent errors

file 2
error:

 12.681950 seconds (230.87 M allocations: 8.952 GiB, 31.95% gc time, 160.44% compilation time)
left hand start:
ERROR: notes too short
Stacktrace:
 [1] split_notes(notes_by_position::Vector{MIDI.Notes{MIDI.Note}}, hand::PianoFingering.Hand)
   @ PianoFingering ~/git-repo/PianoFingering.jl/src/split.jl:7
 [2] splited_range(notes_by_position::Vector{MIDI.Notes{MIDI.Note}}, hand::PianoFingering.Hand)
   @ PianoFingering ~/git-repo/PianoFingering.jl/src/split.jl:79
 [3] run_splite(notes::Vector{MIDI.Notes{MIDI.Note}}, hand::PianoFingering.Hand)
   @ PianoFingering ~/git-repo/PianoFingering.jl/src/process.jl:3
 [4] fingering(file_name::String)
   @ PianoFingering ~/git-repo/PianoFingering.jl/src/process.jl:130
 [5] top-level scope
   @ REPL[13]:1

file 1
error:

  7.029996 seconds (100.48 M allocations: 4.043 GiB, 26.04% gc time, 301.03% compilation time)
ERROR: number of fingering and note not equal.
Stacktrace:
 [1] annotation(part::PyCall.PyObject, fgs::Vector{DataStructures.SortedDict{MIDI.Note, PianoFingering.Finger}})
   @ PianoFingering ~/git-repo/PianoFingering.jl/src/process.jl:85
 [2] fingering(file_name::String)
   @ PianoFingering ~/git-repo/PianoFingering.jl/src/process.jl:137
 [3] top-level scope
   @ REPL[10]:1

Could you create wiki or tutorial, documentation for it to fix stuff better?

[NeroBlackstone]

Thank you for the feedback, I must confess PianoFingering.jl is still in the early stage. Midi is not a support input, because for performance record midi, we don't know whether the notes belong to the left hand or the right hand.
My plan is to train a neural network to separate the left hand and right hand notes, then add fingering for them.