search

bertsky / ocrd_detectron2

OCR-D wrapper for detectron2 based segmentation models
16 stars 5 forks source link

Fix order of import statements for shapely issue #23

Open stweil opened 1 year ago

stweil commented 1 year ago

It's true that such less obvious imports make it difficult to find all affected code locations by a simple code inspection.

But it would still be possible to fix the issue like that case by case.

The current workaround to run pip install --no-binary shapely --force-reinstall shapely has also drawbacks.

bertsky commented 1 year ago

I agree in principle. But we know that core itself, which is used everywhere (CLI decorators), always imports shapely (via ocrd_validators, via ocrd) – before anything else. So that would need to be solved first...