GradientImage and other paint functions not exposed?

[gavbaa]

There are several functions marked as MagicExport in paint.c that don't have a correlating method to get to them in this Go adapter. Is there a reason they're not exposed? Alternatively, is there a way that a user should call them otherwise?

In my particular case, I know that ReadImage("black-white") will call GradientImage, but there's no way to assign artifacts to a MagickWand, and the artifacts needs to exist on the Image itself (setting artifacts on an empty wand will error). But ReadImage creates a new image and then calls GradientImage immediately, so there's no way to set the artifacts using that workflow. So I guess the alternative question is, if GradientImage can't be exposed directly, is there a way to SetImageArtifact such that ReadImage can use them via the Go binding?

[justinfx]

If it is part of the ImageMagick 7 api then it is likely to be something that just wasn't exposed when the bindings were updated to support 7. We can expose missing api that is in the ImageMagick headers

[gavbaa]

It's existed since ImageMagick 6, https://github.com/ImageMagick/ImageMagick/blob/ImageMagick-6/magick/paint.c#L426 . It looks like there might be some other types that will have to be exposed as well, as the method signature includes GradientType, SpreadMethod and PixelPacket.

[justinfx]

Alright. So I guess there is missing v6 api after all depends on when they introduced it in v6. We have PixelPacket defined, but not the others.

…

On Sun, Mar 18, 2018, 1:14 AM George Vilches ***@***.***> wrote: It's existed since ImageMagick 6, https://github.com/ImageMagick/ImageMagick/blob/ImageMagick-6/magick/paint.c#L426 . It looks like there might be some other types that will have to be exposed as well, as the method signature includes GradientType, SpreadMethod and PixelPacket. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#157 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AARPxuMtEkC39GVbUi6DkzTEnSVW6HlHks5tfP47gaJpZM4Suqsr> .

[gavbaa]

I've put up a pull request in #162 that maps this function as it is in IM6. The signature is different for IM7 ( https://github.com/ImageMagick/ImageMagick/blob/master/MagickCore/paint.c#L404 ), but if needed to accept the patch I could explore a version for that branch as well.

[justinfx]

I can try and test this against im7 this weekend

[justinfx]

Was going through these open issues and realised I never followed up on this. I ended up implementing the gradient support for im7. It was definitely a different API, although it has more control since you can have arbitrary color stops.