search

qurator-spk / eynollah

Document Layout Analysis
Apache License 2.0
328 stars 26 forks source link

Eynollah light integration #86

Closed cneud closed 1 year ago

cneud commented 1 year ago

We should start integration of eynollah_light with the main branch - this helps track what conflicts etc. need to be resolved.

cneud commented 1 year ago

Not sure why CI fails, as it works locally, possibly also https://github.com/qurator-spk/eynollah/issues/98?

cneud commented 1 year ago

@vahidrezanezhad: I've started backporting/cherry-picking and testing most of the recent changes from main into eynollah_light.

I did not yet do this for two changes that I would like you to check and adapt/integrate with eynollah_light:

1) remove TF1 session and GC controls, avoid repeating load_model

2) contours: simplify (which implies contours: numpy now needs dtype=object )

Once this is done, we should be able to resolve any remaining conflicts, which would give us a good basis for merging eynollah_light into main.

cneud commented 1 year ago

With 456fccb35e184db1e7a0e73965e446ed8993f41a the CircleCI cache got updated but now both jobs get killed (due to RAM exhaustion?) during the smoke-test 🌧️

~GitHub actions still fails with the old cache~ (Edit: It now works after clearing the cache!)

Locally I get the a working installation with Python 3.7./TF2.11 or Python 3.8/TF2.12 and the 2022-04-05 models in SavedModel format.