mud-pi
mud-pi copied to clipboard
adding list comprehensions
@moonpatroller Hi! Thanks for the contribution :) The list comprehensions certainly make the code more concise (and probably perform better), however I intentionally left them as verbose nested ifs as I thought this might be simpler for beginners to follow. Maybe there's a sweet spot in the middle somewhere? What do you think? :)
Hi! Speaking from the perspective of a person who used this piece of code as a basis of my own project, I certainly appreciate the code simplicity. It helps you grasp basic concepts, especially if someone without Python experience tries to get their teeth stuck in.
In this particular case, I would probably prioritize code simplicity over performance and sophistication - although I will likely be incorporating proposed changes into my own code :)
Best regards
@Frimkron I doubt they make it perform any better. I think beginners have to learn at some point. I see the argument for not constructing complicated lc's, and maybe some or all of these are on the more complicated side. I think I'd be excited to see them when I was learning, but I know lots of students go apeshit over everything, too. This also doesn't seem like a super beginner project. Do as you will. :)