search

OCR-D / ocrd_all

Master repository which includes most other OCR-D repositories as submodules
MIT License
72 stars 17 forks source link

Broken build (ocrd_detectron2) #434

Closed stweil closed 6 months ago

stweil commented 6 months ago

See https://github.com/bertsky/ocrd_detectron2/issues/29 for details. It is broken with any Python version.

bertsky commented 6 months ago

Ah, now I get what's really going on. It's still broken on master, but I could not reproduce elsewhere, because it's a bit fuzzy: it depends on whether kraken or detectron2 get installed first. If Kraken makes it, then due to its torch~=2.1 requirement, we end up with https://github.com/pypa/setuptools/issues/4385.

bertsky commented 6 months ago

No, wait. That CI failure was from your merge. The current master with my #436 is fine.

And the reason this works is because I moved ocrd_kraken to the sub-venv, where it can install its own Pytorch independent of Detectron2.