helio-fm / helio-sequencer

Libre music sequencer for desktop and mobile platforms
https://helio.fm
GNU General Public License v3.0
3.09k stars 115 forks source link

High memory usage + killed when importing midi #287

Closed PowerUser64 closed 11 months ago

PowerUser64 commented 11 months ago

I am getting very high memory usage and a crash when importing a midi file.

Here is a graph of my memory usage while helio is importing a midi file. image

At the spike, you can see helio claim upwards of 25 gb of ram before getting killed by the kernel.

Here is the midi file I imported to caused this: Franz Joseph Haydn - String Quartet in G major - Op. 76 No. 1.zip

peterrudenko commented 11 months ago

Thanks for reporting that! I couldn't reproduce the crash but found some overlooked issues with zero-length sequences (like automations in the attached file), could you check if the development build fixes it for you?

PowerUser64 commented 11 months ago

Just tested and everything seems to work as expected! Thanks for the quick fix!