Robert Sachunsky

@kba you commented > Changes as of 2023-08-20: > > - Try to be less intrusive with OCR-D specific logging conventions to make it easier and less surprising to define...

More explanation on #1080: > Until now, we implemented --log-level/setOverrideLoglevel by changing the default logger to one that does not propagate and overriding the levels of every logger. Now, we...

So perhaps as a workaround we could move the other loggers under `ocrd` (i.e. `ocrd.ocrd_network.*`, `ocrd.processor.*` etc.). But what about other loggers we wanted to affect in ocrd_logging.conf (e.g. `shapely`,...

Also, it seems that the log config file (e.g. `~/ocrd_logging.conf`) still takes precedence: If I run ocrd-tesserocr-recognize with `--log-level ERROR` I can still see `ocrd.workspace.save_image_file` messages with `INFO` level...

> I would tackle #1201 here as well. great! > Currently > > ``` > ocrd resmgr download '*' > ``` > > fails if there are some existing models....

And [this older review](https://github.com/bertsky/core/pull/3#pullrequestreview-887874649) says the culprit is here: https://github.com/OCR-D/core/blob/01955620274e461a8470b37404ef1e9996ca3221/src/ocrd/resource_manager.py#L249-L265 This is not recursive (enough): it cannot copy with subdirectories.

Useful if we want to starting configuring processor module Dockerfiles based on core-cuda-tf1, tf2 or torch instead of just core or core-cuda. (This in turn can be useful for ocrd_all...

Last two commits: I am experimenting with getting builds for other architectures as well. However, further changes are required, because there are usually no OpenCV and lxml binaries available for...

So in b2ae951a628b2a1e24e7c257eba5591d6ef2ccc1, in order to get opencv-python-headless to compile via sdist on arm/v7, which due to `cmake` (as Python package) not being available as a prebuilt binary on that...

> So in [b2ae951](https://github.com/OCR-D/core/commit/b2ae951a628b2a1e24e7c257eba5591d6ef2ccc1), in order to get opencv-python-headless to compile via sdist on arm/v7, which due to `cmake` (as Python package) not being available as a prebuilt binary on...