search

mittagessen / kraken

OCR engine for all the languages
http://kraken.re
Apache License 2.0
688 stars 125 forks source link

Move import statement for KrakenProgressBar from function segtrain to… #557

Closed stweil closed 7 months ago

stweil commented 7 months ago

… segtest

KrakenProgressBar is not used in segtrain, but segtest uses it. Moving the import statement fixes one of several errors reported by flake8.

stweil commented 7 months ago

CI is broken since about 3 months with several errors shown by flake8, see GitHub action.

stweil commented 7 months ago

There remain more issues which cause flake8 to fail.

mittagessen commented 7 months ago

CI is broken since about 3 months with several errors shown by flake8, see GitHub action.

Yes, sorry I've screwed up the merge of the reading order branch and the last months have been extraordinarily busy so I hadn't had the time to bring everything back into order.

There remain more issues which cause flake8 to fail.

They should be fixed now.

stweil commented 7 months ago

They should be fixed now.

Nearly fixed, but there still remain a few errors.

mittagessen commented 7 months ago

On 23/12/11 08:56AM, Stefan Weil wrote:

They should be fixed now.

Nearly fixed, but there still remain a few errors.

Argh, ok. The behavior of the linter changed then. In the past string annotations where a way to circumvent having to import the annotation type as it could lead to circular imports. I guess, not anymore.