generate function in Wrapping Core Image chapter crashes about functional-swift HOT 6 CLOSED

The following solves the bug:

The cropping method isn't really cropping, it adjusts to the size of the to: parameter whether it's larger or smaller than the input image.

CIConstantColorGenerator generates an image with size 0x0, so you need to apply the cropping method before returning it as output of the generate function. That means it needs to use the input image just like the other functions. The following generate function works without crashing in the Xcode 8.2.1 playground:

func generate(color: UIColor) -> Filter {
return { image in
let parameters = [kCIInputColorKey: CIColor(cgColor: color.cgColor)]
guard let filter = CIFilter(name: "CIConstantColorGenerator",
withInputParameters: parameters)
else { fatalError() }
guard let outputImage = filter.outputImage
else { fatalError() }
return outputImage.cropping(to: image.extent)
}
}

Btw the blur function returns an image larger than the input image, so it also needs cropping in the return statement. Once you have added cropping to all the other functions, cropping is no longer needed in the overlay function.

Also to avoid all the "failed to obtain sandbox extension" warning messages I added the following to the Xcode 8.2.1 playground:
URLCache.shared = URLCache(memoryCapacity: 0, diskCapacity: 0, diskPath: nil)

TheFilterType.playground.zip

from functional-swift.

Hi @hugofalkman, I remember we had this problem with old versions UIKit. Could you try on a recent version or on the Mac? It might also be a new problem, not sure!

from functional-swift.

Am on latest Xcode 8.2.1 so am sure it's a new problem. Just copy my code into a fresh Xcode playground to reproduce.

from functional-swift.

Thanks a lot @hugofalkman !

I'm going to leave this open until we have time to do a proper update of the book. That will also help other people stumbling over this issue.

from functional-swift.

This issue was closed, but I don't see an updated version of the book on Gumroad.

from functional-swift.

No, we're working on it =)

from functional-swift.