Robert Sachunsky

Results 735 comments of Robert Sachunsky

BTW, another lead that I had 1yr ago was that in Leptonica's `pngio.c` there were some subtle but significant differences between its `pixReadMemPng` and `pixReadStreamPng` (which IIRC appeared to have...

This appears to be detached from `develop` here. Is this by accident? Maybe @diamond-lizard should make a PR again?

I get something similar: ``` pyrasite --verbose 3298 reverse_python_shell.py b'[New LWP 3309]\n[New LWP 3310]\n[New LWP 3311]\n[New LWP 3312]\n[New LWP 3313]\n[New LWP 3314]\n[New LWP 3315]\n[New LWP 3316]\n[New LWP 3317]\n[New LWP 3322]\n[New...

@kylewang4929 could you please elaborate a little on your changes? 1. 0c6aaad essentially removes the `calloc` for the image struct's data ptr. Your commit message suggests that allocation is redundant...

> 2\. [48c34d2](https://github.com/primetang/pylsd/commit/48c34d22f7072577f71dd8f3ae1459298fd26a84) looks like it exposes all the internal parameters of the LSD library (and standalone CLI) to the C API. Would that enable me to override them via...

> Are you sure you recompiled the library and copied it to `pylsd/lib/linux/liblsd.so`? Duh! Thanks, that's the missing ingredient. I was under the wrong impression `pip install` took care of...

@kba I recommend removing 73648c6 from this PR branch, because upstream would not want to call their package ocrd-fork-pylsd. So could you please remove the latest commit (and force-push) and...

Great to hear this is finally ready. My understanding was that we merge https://github.com/OCR-D/core/pull/678 first and see how this affects RSS (or PSS) througout your test scenarios. Do you have...

> My first question is what your goal is? For `bag -Z -I` it is simply to update the bag checksums (say after some changes to the METS and files)....

BTW, I believe the cause of the misbehaviour might just be the wrong order of the arguments: https://github.com/OCR-D/core/blob/e841ce8443ec7e0fa90e99796b3a947be09844d9/ocrd/ocrd/cli/zip.py#L31-L48 Notice how `directory` and `dest` have been confused.