OpenCV Headless version

[milosacimovic]

Hi Felix,

I was thinking that as OnnxTR is a deployment package it would make more sense to have opencv-python-headless specified in its requirements as it is a more lightweight version.

What do you think?

Kindly,
Milos

[felixdittrich92]

Hi @milosacimovic 👋🏼,

Yeah that's a good idea.

I think two additional install options makes sense cpu-headless and gpu-headless in this case we keep the other both cpu and gpu as is because most other packages relies on the non headless version which would be conflicting.

Wdyt ? :)

[felixdittrich92]

@milosacimovic Opened a PR to add it :)

I would say if we are done with your PR in docTR and it's merged i can sync it to OnnxTR and do a release 👍
Until this i will keep the headless PR - to avoid confiusion with the Readme 😅

[milosacimovic]

Sounds good, I'll finish up the docTr PR first. I should have it by the end of this week. Thanks so much for the reply!

[felixdittrich92]

How does it look @milosacimovic ? ☺️

[milosacimovic]

Hi @felixdittrich92 I've just put in your changes to a new pull request in docTr repo and I've slightly modified the extract_rcrops which works a bit better according to my internal tests.

[felixdittrich92]

@milosacimovic done 🤗
--> https://github.com/felixdittrich92/OnnxTR/releases/tag/v0.5.0 / https://pypi.org/project/onnxtr/

[milosacimovic]

Amazing, thanks @felixdittrich92 !!!