In general:

   - for new question create new email please
   - regarding issue tracker - add there patch. Do not post there code or
   link to code change.

Other comments are below inline...
Zdenko

On Fri, Aug 1, 2014 at 5:52 AM, Tom <[email protected]> wrote:

>
>
> Am Donnerstag, 31. Juli 2014 23:14:30 UTC+2 schrieb zdenop:
>
>> I do not have to time to have a look on this issue yet, but forcing user
>> to use lossless compression is not right way IMO.
>> Right way is to implement option for user to force tesseract to use
>> lossless compression, but this feature is not provided by your "patch"...
>>
>> @zdenop
> @jimregan
>
> Dear zdenop, dear Jim
>
> yes, thanks. I was thinking about an option --force-lossless-compression ,
> but after having inspected the
> http://tesseract-ocr.googlecode.com/svn/trunk/doc/tesseract.1.html
> documentation manual page, I think, that Tesseract does not support (apart
> from a few) command line options, Instead, it (mainly) supports to have
> options in a config file.
>

Thanks for heads up - man page was not updated for a longer time. If you
run "tesseract --help" (in 3.03 version) you can see there much more
options. Nick implemented to possibility to define control parameter from
command line:

 -c configvar=value    set value for control parameter. Multiple -c
arguments are allowed.


> So I will modify my code so that lossless compression can be forced by
> enabling it by means of a switch in the config file.
>
> Question 1
> ========
>
> Please can you let me know, if you like my approach (config parameter), or
> if you would also support my proposal for a command line switch
> (--force-lossless-compression).
>

IMO you need to implement "config parameter" if you want to create command
line switch (that is not necessary as you can pass config parameter at
command line).
Create it that way so user can force type of encoding (jpeg, g4, flate - it
would be great if leptonica will support there also jbig2[1] ;-) ) As a
default option leave the current behavior (so selectDefaultPdfEncoding
choose the type of encoding)

[1] https://github.com/agl/jbig2enc


> BTW, it was and is clear to me, that a final patch must not contain
> out-commented (dead) code.
>
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"tesseract-ocr" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/tesseract-ocr.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/tesseract-ocr/CAJbzG8xirkWY43ZPW7FbR-tu5oNYRNZN3C%2Bg0uZACH-bv5W0cQ%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to