thkoeln / dlaproject

0 stars 0 forks source link

Manche Stücke können nicht konvertiert werden #7

Closed pkalytta closed 3 years ago

pkalytta commented 3 years ago

https://github.com/thkoeln/dlaproject/blob/eff514cadc7cb2a08f0860fcfab64a6a2673ca0c/src/datasets/MidiParser.py#L108

Das Stück /datasets/midi_originals/balakirew/islamei_format0.mid kann nicht konvertiert werden, da es scheinbar keine zweite Hand gibt?

File "\PycharmProjects\dlaproject\src\datasets\MidiParser.py", line 108, in midiToArray leftPart = midif.parts[1] File "\AppData\Local\Programs\Python\Python38\lib\site-packages\music21\stream\iterator.py", line 336, in __getitem__ e = fe[k] IndexError: list index out of range

pkalytta commented 3 years ago

generell ist die frage, ob es nicht vllt midi-stücke gibt, die quasi nicht die zwei-hand regel befolgen, vllt sollte dieser code dynamisch für eine beliebige anzahl an "händen" ausführbar sein. Lässt sich die anzahl an "parts" in midif herausfinden?

henrikerp commented 3 years ago

Richtig, die ersten Zwei Tracks hardcoded auszuwählen soll nur temporär sein, besser man untersucht welche Parts Noten enthalten und welches Instrument gespielt wird, und wählt anhand dessen aus.

henrikerp commented 3 years ago

Sollte sich ja jetzt hoffentlich erledigt haben.