66
submitted 10 months ago by Vitaly@feddit.uk to c/programmerhumor@lemmy.ml
top 15 comments
sorted by: hot top controversial new old
[-] independantiste@sh.itjust.works 21 points 10 months ago

Took 4 months to what? Download this image?

[-] Vitaly@feddit.uk 6 points 10 months ago
[-] nothacking@discuss.tchncs.de 7 points 10 months ago

And in the end you turned it back to an image.

[-] Vitaly@feddit.uk 1 points 10 months ago

because tiles like this are not supported on lemmy

[-] xmunk@sh.itjust.works 6 points 10 months ago

It's always fun building interesting things, good job!

[-] Vitaly@feddit.uk 4 points 10 months ago
[-] darcy@sh.itjust.works 3 points 10 months ago

i read through the code, and i have some tips, which you may find helpful (or not):

  • argument parsing: if --color is not the first argument, it will try to open a file named --color, which i assume is unintended. i would suggest checking out the clap crate to easily parse args
  • i'm quite sure why you used 'clusters' instead of resizing the image to the terminal width? if it is purely as a programming exercise, or for performance reasons, then that's great! but otherwise, calling image.resize() is easier
  • .len() on a string returns length in bytes, not characters, so could break with non-ascii text. in the context of this program, the text will always be ascii, so it is of course not a problem, but it's worth to keep in mind. to get character length, use .chars().count()
  • in my testing, the width of the image is always affected by the width of the terminal, always being less than the maximum possible width, causing the image to be stretched vertically. i'm not sure why this is happening
  • in get_brightness_of_cluster, pushing to a Vec and then calling .sum() can be replaced with a loop which increments a mutable u32 variable. this is a nitpick, but it can avoid unnecessary memory allocation

check out this example. sorry if this comes off as rude or a nitpick, i'm just trying to provide some advice :)

[-] Vitaly@feddit.uk 3 points 10 months ago

Thank you for your help this is not rude at all and very helpfull! I know about argument parsing with clap, and started thinking about it as I started to add more functional to the program

Is resizing and image is more performant? I yes then i was wrong when coding with clusters, it's always good to have another viewpoint for tasks like this I didn't even think of resizing an image)

About .len i think it's fine as long as it works

The width of images should be resized depending on your terminal width and height, im still not sure how i can improve it, because every image is unique

get_brightness is a bit broken in performance right now, i will fix it later

Thanks you for your tips, really helpfull! Дякую!

[-] darcy@sh.itjust.works 1 points 10 months ago

:) no problem! i would assume resizing the image might be a little slower, because it creates a clone of the image, but if you use FilterType::NearestNeighbor, the speed is negligable in my opinion

[-] MrPoopyButthole@lemmy.world 3 points 10 months ago

What languages did you come to rust from?

[-] Vitaly@feddit.uk 12 points 10 months ago

Python... you can guess by my shitty code

[-] blotz@lemmy.world 2 points 10 months ago

Very cool! Did you consider any other interpolation algorithms?

[-] Vitaly@feddit.uk 1 points 10 months ago
[-] blotz@lemmy.world 12 points 10 months ago* (last edited 10 months ago)

Its the mathematical term for estimation and constructing new data from existing data. In the context of what you are doing, it's resizing images.

You are doing something called linear interpolation. This works great for shrinking an image. However, have you considered what happens when your ASCII resolution is greater than your image resolution? This is where bilinear and bicubic interpolation come in.

These algorithms are cool but are also massive overkill for your average use case. They only make a different in a very niche use case (when your ASCII resolution is greater than your image, such as pixelart)

Great code! Very cool

[-] Vitaly@feddit.uk 2 points 10 months ago

thank you so much! have a good day!

this post was submitted on 27 Oct 2023
66 points (79.5% liked)

Programmer Humor

32045 readers
1416 users here now

Post funny things about programming here! (Or just rant about your favourite programming language.)

Rules:

founded 5 years ago
MODERATORS