djask / bms_converters

Convert osu!mania to BMS BMSON (with basic SV Support)
1 stars 0 forks source link

Issues with sanitise_event() #2

Open hadronyche opened 1 year ago

hadronyche commented 1 year ago

Do video events usually have (x, y) values? I found one without:

//Background and Video events
Video,0,"NisekoiOP2.avi"
line 117, in sanitise_event
    ret["x"] = int(e[3])
IndexError: list index out of range

I'm not sure what these events are for (Background Colour Transformations?) but there's an append before declaration:

in sanitise_event
    ret[f"param{idx}"].append(p)
KeyError: 'param0'
djask commented 1 year ago

Can't handle video data for now anyway (should be easy to add) so just ignoring video type events should solve this,