baranyildirim / EvoGAN

EvoGAN: Evolutionary Algorithm based Neural Architecture Search for Generative Adversarial Networks
MIT License
9 stars 3 forks source link

Why isn't list comprehension used? #3

Open sepehr78 opened 2 years ago

sepehr78 commented 2 years ago

In the _evotrain.py file there is a function called _generate_newdna where a for loop is used instead of list comprehension. I wanted to inform you that list comprehension is MUCH slower and less readable than for loops. See here.

I really like this project and want to use your code, but the fact that you used a for loop is honestly despicable. Please fix it and I will use your code and star it.