Robert Sachunsky

Results 721 comments of Robert Sachunsky
trafficstars

@clavelc, most of what you say is not related – please help keeping issues to the point! > I have to clean up with sed to delete all [ |...

Fixed in https://github.com/cisocrgroup/ocrd_cis/pull/87/commits/5282092997ad6b2b53a3cef8c3c96fbb27066682 (part of https://github.com/cisocrgroup/ocrd_cis/pull/87 shipped with ocrd_all).

BTW, I just tested under (METS Server and) `OCRD_MAX_PARALLEL_PAGES=2` – it works, but you need lots of GPU memory, otherwise GPU OOM happens. (It does work with `CUDA_VISIBLE_DEVICES=`, but of...

Do we really need to run CI on the Docker images? Seems redundant... (Also, it would require cloning the repo and mounting tests and test data into the container...)

The namespace problem has ultimately been fixed by #133, so this should be closed. (And #151 will bring a functional replacement version of ocrd-sbb-binarize.)

> Since this is going to populate the produced Nextflow files with undesired outputs, I have to back up by deep copying the parameters and restoring them back after the...

> However, I have one main issue with Task Sequence - it validates workflow data during run-time (which also requires access to the Workspace mets). The idea is to validate...

Even worse, in my case (ocrd==c8aef091e18ce4e79cb3a3b7399ac908c90a169d), the processor is already installed, but resmgr crashes: ocrd resmgr download ocrd-tesserocr-recognize frak2021.traineddata ``` Traceback (most recent call last): File "/data/ocr-d/ocrd_all/venv38/bin/ocrd", line 8, in...

Indeed, this is hideous. Using MANIFEST.in instead of setuptools discovery and exclusion mechanisms is not only tedious, but quite unexpected. In that sense related: https://github.com/pypa/setuptools/issues/3260