CatchTheTornado / CatchTheTornado/text-extract-api

[bug] Strategy cache hash should not depend solely on the file itself

Open
#96 1 comment 1 reaction 0 assignees View on GitHub
bug
Dominant language
Python
Stars
3.2k
Forks
279
PR merge metrics
No merged PRs in 30d

Description

Changing the strategy in request obviously doesn't change anything when the cache is used (even if OCR was performed using a different strategy).

- [ ] Analyze which fields could critically interfere with the results (I guess it's `file_hash + strategy_name + language`).
- [ ] Combine them (it's not necessary to hash them after proper validation; it could be `language:strategy_name:file_hash`).

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.