-
Notifications
You must be signed in to change notification settings - Fork 250
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
UIImagecategories 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 domemcmpbefore it calculates the percent.Fixes #109