search

Diaoul / subliminal

Subtitles, faster than your thoughts
http://subliminal.readthedocs.org
MIT License
2.41k stars 313 forks source link

Too much time is spent here according to profiler. #909

Closed gberes01 closed 5 years ago

gberes01 commented 6 years ago

https://github.com/Diaoul/subliminal/blob/dd74383d1cba82829ce720f2e439a65d13ffe7ef/subliminal/utils.py#L68

Hello. I've tried to run subliminal for a large library and concluded that it is way too slow. Profiler shows that about 90% of the CPU time is spent here. Is it really necessary to generate the hash on 10MB data? If so, could we just leave napiprojekt out from the default configuration?

ratoaq2 commented 5 years ago

You're right. Subliminal could just compute hashes for the enabled providers

ratoaq2 commented 5 years ago

Hashes are only computed for requested providers Fixed by eec2c3211fe876159ccc3f80e43dc5c0bf91191c