search

fourMs / MGT-python

Musical Gestures Toolbox for Python
https://www.uio.no/ritmo/english/research/labs/fourms/downloads/software/musicalgesturestoolbox/mgt-python/index.html
GNU General Public License v3.0
52 stars 11 forks source link

blur_faces() error 2 #308

Closed alexarje closed 10 months ago

alexarje commented 10 months ago

The latest fix of blur_faces() removed the progress bar but introduced a new error (2)... The blurred video is saved correctly, but it chokes on saving the csv file. Screenshot_20231114_205321

joachimpoutaraud commented 10 months ago

When fixing the previous issue with the blur_faces(), I did some minor adjustements to the embed_audio_in_video function, but there is apparently still something I am not able to understand properly.

There are parts of the code in the toolbox which are a bit redundant or which do not use functions already available in the toolbox. At some point, it would be great to meet with Balint to do some recap of all the functions available in the toolbox, in order to see if we need to keep them all or do some cleaning (especially in the _utils.py script).

For now, I have fixed the issue related to the progress bar and removed what I did on the embed_audio_in_video function.