search

ELITR / SLTev

SLTev is a tool for comprehensive evaluation of (simultaneous) spoken language translation.
8 stars 3 forks source link

SLTev fails to delete temporary directory and reports mwerSegmenter failure #70

Open bhaddow opened 2 years ago

bhaddow commented 2 years ago

Hi

When I run SLTev, it fails, reporting that mwerSegmenter is missing. After some digging, I realised that mwerSegmenter is fine. The problem occurs at this line:

https://github.com/ELITR/SLTev/blob/2d56dd08fc314ba819eec5dcaaae7fc3c86c8559/SLTev/files_modules.py#L214

The directory deletion fails (not sure why, could be an nfs problem) an exception is thrown, caught and turned into a mwerSegmenter error. Setting ignore_errors=True in shutil.rmtree(temp_folder) is a temporary fix, but I think the real fix is to use python's tempfile module to create a temporary directory.

A further problem is that catching the error, and outputting a generic mwerSegmenter message, makes this harder to debug

best Barry