Makie.jl icon indicating copy to clipboard operation
Makie.jl copied to clipboard

don't remove underlying VideoStream file when doing save()

Open aplavin opened this issue 1 year ago • 3 comments
trafficstars

Description

For some reason, the file was removed when calling save(path, ::VideoStream). This breaks all following show calls: they, clearly, rely on the video file still being present.

Type of change

Delete options that do not apply:

  • [x] Bug fix (non-breaking change which fixes an issue)

aplavin avatar May 23 '24 19:05 aplavin

Hm, that was a lazy clean up, assuming that one records a video and then when saving it, it isn't needed anymore. I think we'll need to move this into a finalizer for VideoStream to not have this accumulate.

SimonDanisch avatar May 24 '24 10:05 SimonDanisch

Hm, I don't think Julia has finalizers for regular structs... Should we make it mutable struct then?

aplavin avatar May 24 '24 11:05 aplavin

Yes, I guess we'll need to do that then ;)

SimonDanisch avatar May 24 '24 12:05 SimonDanisch

bump... would be nice to fix this

aplavin avatar Aug 01 '24 22:08 aplavin

Thank you!

SimonDanisch avatar Aug 09 '24 08:08 SimonDanisch