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 :)
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
:) 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
Absolutely! The original focus on having a black cat day specifically included black-and-white cats.
“When the campaign was launched, statistics revealed that black and black-and-white cats took, on average, seven days longer to find a home compared to cats of other colours”
I’m hoping to make it healthy enough that the flying fuckwits (presuming you mean mosquitoes) are kept under predation. We already have a mosquito problem via our back garden as a neighbour has a (I think) neglected pond. Might try and help them get it sorted if all goes well.
We’re also not far from a fairly small and sluggish river so… ¯_(ツ)_/¯
You’re not wrong, but those flying fuckwits are an important and valid part of the ecosystem. Rewilding is all about doing away with some of our human conveniences that have made life difficult for all types of wildlife.
feddit.uk
Top