Gameslinger / rust-stereogram

A random-dot autostereogram generator written in rust
1 stars 0 forks source link

POSIX style arguments #1

Closed Goreek closed 8 months ago

Goreek commented 8 months ago

I like the project very much (stereograms are really underrated), and I'm also learning Rust.

I suggest to implement POSIX style CLI arguments for the utility. Obvious proposal is:

--depth-source, -d Image file name that represent depth distribution. --pattern-size, -p Size of repeating dots pattern square (in pixels) --shift-amplitude, -s Amplitude of distortion of repeating pattern to represent depth (0.1 to 0.9) --width, -w Width of outer image (in pixels) --height, -h Height of outer image (in pixels)
--out,-o Stereogram output image file name

So utility will be called ./autostereogram --depth-source=shark.png --pattern-size=40 --shift-amplitude=0.6 --width=600 --height=400 --out=shark-stereogram.png

All parameters except --depth-source and --out can be ommitted, with default value applied. For --width and --height default value should be just the same size as original depth image.

I can do it, if you assign the task on me.

Gameslinger commented 8 months ago

I'm glad you like the project! That is a great idea and improvement. I assigned you to the issue. Have fun!

Goreek commented 8 months ago

It looks like I don't have permissions to create a branch for the issue. Please, provide it to me or create the branch posix-cli-args at the right panel in Development section.

Gameslinger commented 8 months ago

Thank you for your patience. Hopefully, you are able to contribute now.

Goreek commented 8 months ago

Hmmm, I suppose I still can't contribute until you give me such permission for the repository.

~\projects\repo\rust-stereogram>git push remote: Permission to Gameslinger/rust-stereogram.git denied to Goreek.

Please try to add me at the \<Repository main page> -> Settings ->Collaborators

Goreek commented 8 months ago

I accepted you invitation, and now I can submit. Thank you!

Gameslinger commented 8 months ago

Glad we got that figured out! Thanks for contributing!