notrobro icon indicating copy to clipboard operation
notrobro copied to clipboard

Outro matching only seems to support one method in the detector

Open razzeee opened this issue 6 years ago • 2 comments

razzeee avatar Aug 05 '18 11:08 razzeee

This was because outros frames were already very less in the majority of the shows and the second method i.e. 'longest common subsequence' gave even lesser matches as compared to 'all matches'. So didn't make much sense to add the other method.

mohit-0212 avatar Aug 05 '18 14:08 mohit-0212

Then we should investigate if we need to keep longest_common match. It might not be feasible to keep that method around.

razzeee avatar Aug 05 '18 15:08 razzeee