wireviz / WireViz

Easily document cables and wiring harnesses.
GNU General Public License v3.0
4.32k stars 220 forks source link

Add option to being able to perform basic transformation on images (rotate/mirror) #421

Open martonmiklos opened 1 week ago

martonmiklos commented 1 week ago

While working on #418 I got an idea: I used to create images in left and right variants for different connector orientations: kép kép

It would be great if the wireviz would provide option least for horizontal mirroring of the images. If we there at the implementation rotating vertical mirroring too.

Comments, concerns, ideas welcome in this topic!

kvid commented 1 week ago

I do see the usefulness in such simple image processing (mirror, rotate, and maybe crop), and it isn't too hard to do each of them, but we also need to create and write the result to a temporary image file with a name that doesn't conflict with other names reachable through the image_paths - and optionally delete it again in a final clean-up step when no longer needed. A similar need is also described in https://github.com/wireviz/WireViz/issues/322#issuecomment-1640323088 - a common code should be created.

If more than one processing step is allowed, then their order must be specified.

Do you know any other additional image processing steps that might be requested in the future?