Robert Sachunsky

Results 735 comments of Robert Sachunsky

BTW, is there a particular reason for keeping the TF1-style session management? I found that if I remove it completely (including the explicit GC calls), **and** avoid repeating `load_model` calls...

I don't know about other use cases (plotting / cropped image is not used by OCR-D), but to wrap the cropper into a separate processor, I think we do need...

Indeed, I cannot reproduce myself anymore. I believe the last version was 0.0.11, so I assume this has been fixed at some point.

Having a dedicated CLI command for this opens up the opportunity to give it an optionally different behaviour: downloading _without changing the refs_! (So essentially, this could just do what...

Another idea I had: couldn't we at least try to force PIL.Image objects we ourselves produce (`Workspace.image_from_page` and `image_from_segment`) to retain or piggy-back the DPI info? Or even store that...

> Most OCR-D processors return 1 when they are called without any argument, but `ocrd` from core is an exception which returns 0. Should that be changed? I don't think...

> Where is it documented that running `ocrd` without an argument should return 0? That's click's default (because traditionally that's how multi-task CLI behave). > If there is no such...

fixed [long ago](https://github.com/OCR-D/ocrd_all/commit/d53b5b00fc4d27ad1872e38edfbeef283e65a0b7)

Also: limiting **CPU-side RSS** would be useful. We could do this from the outside (on the [`docker run` cmdline](https://docs.docker.com/config/containers/resource_constraints/) or in [docker-compose.yml](https://docs.docker.com/compose/compose-file/deploy/#devices)) – e.g. `--memory 2G` or `--ulimit rss=2000000:4000000` (resp....

> `/etc/profile.d` might not work as expected; I believe it's only sourced for login shells, Indeed. For non-interactive bash, one can pass `BASH_ENV` to be sourced, but when invoked as...