Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve Image Quality for "Resizable Custom Crop" #12

Closed
gekitz opened this issue Mar 10, 2013 · 10 comments
Closed

Improve Image Quality for "Resizable Custom Crop" #12

gekitz opened this issue Mar 10, 2013 · 10 comments

Comments

@gekitz
Copy link
Owner

gekitz commented Mar 10, 2013

As the image quality of the custom crop was improved with the latest commit 2a91f31 we should definitely improve the quality of the "Resizeable Custom Crop" too.

@Pathonhauser, any ideas on that?

@phausero
Copy link
Contributor

Yap. Work in progress...

Sent from Mailbox for iPhone

On Sun, Mar 10, 2013 at 7:24 PM, Georg Kitz notifications@github.com
wrote:

As the image quality of the custom crop was improved with the latest commit 2a91f31 we should definitely improve the quality of the "Resizeable Custom Crop" too.

@Pathonhauser, any ideas on that?

Reply to this email directly or view it on GitHub:
#12

@jamgraham
Copy link

Any ETA on this? Thanks!

@gekitz
Copy link
Owner Author

gekitz commented Mar 28, 2013

@pathonhauser any news on that?

@convey-mane
Copy link

Well ... 5 month now ...
My problem: I solely use this Resizeable Custom Crop (even stripped out everything else).
As we "scan" businesscards (take photos on e.g. iPhone5) to send 'em to a server for OCR, we on one hand want to crop all non-text (background, holding fingers and such) to reduce bandwidth/upload time - on the other hand, sent data has to be of high quality to get reasonable OCR-results.
And with an example-file of 399 KB uncropped resulting in 93 KB with Resizable Custom Crop - instead of 350 KB when test-cropping to the same frame with that simple Preview app on Mac (is it named "Preview"? In German it's "Vorschau"), there's not much left to OCR.
My suspicion is, it's the display-size, because on an iPad retina the difference between photo-resolution (well, indeed I use video-resolution at the moment, but that's another story) and display-resolution is minimal, resulting in good quality crops (a photo has e.g. 432 KB, with the crop still at 419 KB).

So, before I dive into GKImageCropView.m's croppedImage: is there already work done, will there be a solution anytime soon?

Greetings

@convey-mane
Copy link

found out something else, it apparently makes a huge difference in how the crop is done:

  • if it's done by narrowing the cropping-rectangle (and maybe the image isn't scaled), or
  • if the image is zoomed up, but the cropping-rectangle isn't touched (like with the other cropping methods);

the former results in the noticed poor quality, the latter not.
(with a test-image of 185 KB - it's made with my old (non-retina) iPad2 - the first crop is only at 63 KB, but the second is still at 126 KB, which seems to be a similar result as cropped with Preview app)!

So maybe it's not the display size ... but the scale of the cropping rectangle.

@phausero
Copy link
Contributor

Hi there,

Sry that it took so long. I just took a look at it and i think I found a solution. I'm going to make a pull request on the weekend because I'm really busy right now and I want to test it in depth. Can you wait till then or do you want me to post the code right here and you have to test it on your own?

@convey-mane
Copy link

Well, as the colleague, which is doing the Server-side OCR-part also still has something to do - and is in holidays this and next week, I think it would be OK, when there's a tested solution on Monday, then I still can finish my part before he's back :-)
And with that also my boss will be satisfied 'till next week :-))
This week I have something else to finish anyway.

Very Fine.
Thanks a lot for the fast answer and (hopefully) the coming solution!

phausero added a commit to phausero/GKImagePicker that referenced this issue Aug 23, 2013
@convey-mane
Copy link

What could I say ... hats off, great, brilliant, ...!!!
:-)
Thanks a lot!

@phausero
Copy link
Contributor

Great to hear that you like it! :-)

@gekitz
Copy link
Owner Author

gekitz commented Aug 29, 2013

I think we can close it now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants