Skip to content
This repository has been archived by the owner on Oct 28, 2023. It is now read-only.

Suggestion: CLI option to automatically add mirrorings and rotations of input image #30

Closed
vi opened this issue Sep 13, 2019 · 2 comments · Fixed by #57
Closed

Suggestion: CLI option to automatically add mirrorings and rotations of input image #30

vi opened this issue Sep 13, 2019 · 2 comments · Fixed by #57
Labels
enhancement New feature or request Hacktoberfest help wanted Extra attention is needed

Comments

@vi
Copy link

vi commented Sep 13, 2019

Suggesting to have something like --fips-and-rotates option:

texture-synthesis -o out.png --tiling generate --fips-and-rotates in.png

Which does approximately this:

convert in.png -rotate 0 in1.png
convert in.png -rotate 90 in2.png
convert in.png -rotate 180 in3.png
convert in.png -rotate 270 in4.png
convert in.png -rotate 0 -flip in5.png
convert in.png -rotate 90 -flip in6.png
convert in.png -rotate 180 -flip in7.png
convert in.png -rotate 270 -flip in8.png
texture-synthesis -o out.png --tiling generate --fips-and-rotates in*.png
@vi vi added the enhancement New feature or request label Sep 13, 2019
@Jake-Shadle Jake-Shadle added the help wanted Extra attention is needed label Sep 16, 2019
@Jake-Shadle
Copy link
Member

This seems like a cool feature to have for the CLI, but I don't think we're planning on doing this in the short term, but PRs are of course welcome!

@vi
Copy link
Author

vi commented Mar 2, 2021

Although there is now the flip-and-rotate subcommand, it only works in the most primitive mode (i.e. specify a style and get some picture), not in guided mode.

I expected that --flips-and-rotates flag also automatically flipped and rotated the associated --guides in sync with example pictures.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request Hacktoberfest help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants