-
Notifications
You must be signed in to change notification settings - Fork 253
Report precent difference when using a tolerance #149
base: master
Are you sure you want to change the base?
Conversation
CGImageGetBytesPerRow(self.CGImage), | ||
CGImageGetColorSpace(self.CGImage), | ||
(CGBitmapInfo)kCGImageAlphaPremultipliedLast); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice, remove extra line
so good! Some nit's here and there but thank for working on this. |
1d1540d
to
581dac6
Compare
Updated the commit w/ the nits taken care of. |
517bb74
to
5e74d9c
Compare
Actually I just realized (by noticing a warning I missed) why that float was initialized. The compiler warns that |
@ultramiraculous yes you are right, |
5e74d9c
to
066354d
Compare
@nscoding, yup I switched it to |
Thank you for your pull request. As you may know, we require contributors to sign our Contributor License Agreement, and we don't seem to have you on file and listed as active anymore. In order for us to review and merge your code, please email [email protected] with your details so we can update your status. |
1 similar comment
Thank you for your pull request. As you may know, we require contributors to sign our Contributor License Agreement, and we don't seem to have you on file and listed as active anymore. In order for us to review and merge your code, please email [email protected] with your details so we can update your status. |
@nscoding I completely missed that this PR was still open. You should tell @jamesgpearce the bot is being spammy, but in this case I guess it helped me remember it. Is it still worthwhile to get this passing? |
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
When using a tolerance, report the % difference to the console on failure + add a key to the NSError with the percent.
I had to refactor the
UIImage
categories to separate out the part where we calculate the percent difference. As-written it might be a little faster if the images are the same when using a tolerance, because it tries to domemcmp
before it calculates the percent.Fixes #109