search

Happy-Algorithms-League / hal-cgp

Cartesian genetic programming (CGP) in pure Python.
GNU General Public License v3.0
28 stars 10 forks source link

Set reorder=True per default. #360

Open HenrikMettler opened 2 years ago

HenrikMettler commented 2 years ago

Moves the boolean reorder_genome into population to check whether genome conditions (n_rows=1, levels_back=n_columns) for reorder=True are met. I am not removing the possibility to configure this, since reorder is limited to certain genome parameter configurations. Closes #347.

jakobj commented 2 years ago

good catch that we can not just remove the configuration option for reorder while we support multiple rows.

how do you feel about removing the possibility of setting n_rows and always use n_rows=1? my impression is that modern general purpose CGP anyways uses exclusively only one row but may columns, while multiple rows were historically used for evolving circuits.

HenrikMettler commented 2 years ago

yeah, i think I agree. Should we also remove levels_back? Atm I can't see any use case where anything but levels_back=n_columns is useful.

jakobj commented 2 years ago

yes, i think you're right, levels_back can also be removed. :+1:

could you please create a separate PR, removing n_rows and levels_back? after that has been merged, we can come back to this one.