Robert Sachunsky

Results 721 comments of Robert Sachunsky
trafficstars

Not so sure about the time frame for this though. Since it involves patching all modules, I guess the final workshop is out of the question. Setting `3.0` to take...

> Since it involves patching all modules If we just assume the derived image passed to `save_image_file` must come from the (cropped/deskewed/dewarped/binarized/denoised/...) original _without rescaling_ somehow, then we'll only need...

> To clarify: For a PAGE URL `https://remote/page.xml`, you want to download the PAGE-XML and then resolve the `Page/@imageFilename` / `AlternativeImage/@filename` references by prepending `http://remote` to the file paths? No,...

> Without this call any logging records in any other application that uses stuff from `ocrd-core` gets dropped. Why become all loggers silently `PropagationShyLoggers`? Sorry to hear this hit you...

> If the main reason to turn off logging totally is the behavior of tensorflow, question: It is possible to integrate tensowflow into the default logging conf file? No, not...

Should probably differentiate ALTO versions, though.

Indeed, very annoying. It's not the OCR-D logging, but stdout will quickly fill with megabytes, even in scripts. I think the best fix would be to follow [Click's documentation](https://click.palletsprojects.com/en/7.x/api/?highlight=progressbar#click.progressbar) and...

> I don't know what would be the right thing to do: Fix the docstring or fix the implementation? I'd prefer the latter option.

This will conflict with #652 (where that part of `run_processor` is delegated to the new `run_api`) – not sure what's the least painful way for merging. Oh, since I saw...

> > This will conflict with #652 (where that part of run_processor is delegated to the new run_api) – not sure what's the least painful way for merging. > >...