imageWithPerspectiveCorrectionFromQuad corrupts source image about aggeometrykit HOT 6 CLOSED

Have you seen this pull request? gavrix@4bd09ed

Also, we know that the resampling could be improved (#23). Currently we are using nearest neighbor which results in a bit more pixelated images than necessary.

from aggeometrykit.

Nope. But I've made almost the same changes in the code. It's good for development purposes, however due to company policy I just can not use pods with my modifications. Any chance to see an updated version with that fix any time soon?
As for resampling it is not an issue for me but it's good to know you're working on it!
P.S.: btw, AGGeometryKit is awesome!

from aggeometrykit.

^ ^, That's a good company policy. I support that! You don't want changes like that discarded on each pod install.

That pull request is live on master and published as version 1.2.3. A pod update should normally be enough.

We're sadly not working on improving the resampling. I don't have time for that, but I hope others would like to contribute on that part. It will be very interesting, fun and educational.

from aggeometrykit.

Ahhh, man. I just needed to update the pod. I was using 1.2.2. Sorry for wasting your time!
Thanks!

I've just noticed an issue with UIImage scale and "imageWithPerspectiveCorrectionFromQuad". If I pass an image with scale factor 3.0f (iPhone6 Plus) it crops only portion of an image. As far as I traced the rects it appears to act weird here:

(Line 164:) UIImage *correctedImage = [self imageWithTransform:transform anchorPoint:CGPointZero];
(Line 165:) UIImage *resultImage = [correctedImage imageByCroppingToRect:destinationRect];

Here, imageWithTransform returns an image with scale 1.0f (so its size in points = size in pixels) and destinationRect is still in points. So the result image is 1/3 upper part of the source image.

Is this something you're aware of and I'm wasting your time again?

from aggeometrykit.

No problem! :) 1.2.3 was released very recently – good timing.

I'm not aware of this. A pull request on the scale issues would be highly appreciated.

from aggeometrykit.

And also perfect if you can open a separate issue for that.

from aggeometrykit.