r-rust / gifski

R wrapper for the 'gifski' Rust Cargo crate.
Other
73 stars 4 forks source link

Add a 'backward' option #26

Closed stla closed 1 year ago

stla commented 1 year ago

Hello,

I think it would be nice to have a 'backward' option, allowing to record the GIF from frame 1 to frame n to frame 1 (as I did here). It's possible with ImageMagick, but the author of gifski doesn't want to add this option (I already asked him).

jeroen commented 1 year ago

I don't understand, how is this different from passing rev(images) in your input?

stla commented 1 year ago

Sorry, I mean forward and backward. I'm not sure this is the same as passing c(images, rev(images)) because the gifski command takes the images in lexicographical order of their names, but maybe the gifski package takes the given order of the images?

jeroen commented 1 year ago

Yes we just loop over the input images in the provided order:

https://github.com/r-rust/gifski/blob/850797a1bb5d4536cc79dec278b18ffaba43b95a/src/wrapper.c#L35-L41

stla commented 1 year ago

Ok, sorry. Thanks.