search

mozilla / DSAlign

DeepSpeech based forced alignment tool
Mozilla Public License 2.0
235 stars 33 forks source link

Do not load model if it's already loaded #27

Open gibiansky opened 4 years ago

gibiansky commented 4 years ago

Not only is this slow, this will break on GPUs because TensorFlow does not release its allocated memory. So if you try to run a catalog with multiple files on a GPU, the first file will succeed, and the second file will give you an OOM error.

gibiansky commented 4 years ago

I'm not totally sure how this interacts with multiple processes though, or if this creates some sort of race condition. On first glance I think it's fine but I haven't thought about this deeply.

I made this PR because I hit this issue and the change I put in this PR fixed it so figured I'd put this up in case someone else hits this.