Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Composite with x,y position as array #39

Closed
stefanak-michal opened this issue Jun 27, 2019 · 2 comments
Closed

Composite with x,y position as array #39

stefanak-michal opened this issue Jun 27, 2019 · 2 comments
Labels
enhancement New feature or request
Milestone
1.1.0

Comments

@stefanak-michal
Copy link

Hi.

Composite method supports more than one picture. But offset as x,y position is only integer. It will be nice, to have a opportunity to insert x,y as arrays.

https://libvips.github.io/libvips/API/current/libvips-conversion.html#vips-composite
x : VipsArrayInt, position of subimages
y : VipsArrayInt, position of subimages
@stefanak-michal stefanak-michal mentioned this issue Jun 27, 2019
Closed
kleisauke pushed a commit that referenced this issue Jun 27, 2019
@kleisauke
Change the composite x and y positions to array (#39)
511c70e
kleisauke added a commit that referenced this issue Jun 27, 2019
@kleisauke
Changelog entry for #39
2e5fa14 
+ update dependencies and some small cleanups.
@kleisauke kleisauke added the enhancement New feature or request label Jun 27, 2019
@kleisauke kleisauke added this to the 1.1.0 milestone Jun 27, 2019
@kleisauke
Copy link
Owner

Indeed, it looks like a little oversight. Thanks for the PR and reporting this.

Fixed in commit 511c70e to be included in v1.1.0.

@kleisauke
Copy link
Owner

NetVips v1.1.0 is now available.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
1.1.0
Development

No branches or pull requests
2 participants
@stefanak-michal @kleisauke