icatcherplus / icatcher_plus

iCatcher+: Robust and automated annotation of infant gaze from videos collected in laboratory, field, and online studies
GNU General Public License v3.0
17 stars 19 forks source link

Sw/fix crashes #66

Closed samwinebrake closed 1 year ago

samwinebrake commented 1 year ago

These updates relate to the crashes occurring during parallel processing. Now, the available RAM of the user is checked and a default value of 75% of that is used to create batches of frames that are processed and sent off into parallel processes. This percentage was added as a flag that the user can change. (Also, feel free to change the default value if 75% is too high/low).

yoterel commented 1 year ago

Thanks ! I am hesitant to merge this feature for now, as the default flags for CPU-track was already changed to not buffer. I consider this a solution to an edge case (users who have no GPU, not enough RAM, do not want to use the faster models, and can't tolerate the running time without batching) which involves estimating RAM through psutils...

If users have such use-case, they can pull this PR themselves and add the code. Or open an issue and I will reconsider...