Simple_GA: Wrong decay parameter ("sigma_limit" instead of "sigma_decay")

RobertTLange / evosax

Evolution Strategies in JAX 🦎

Apache License 2.0

479 stars 44 forks source link

Simple_GA: Wrong decay parameter ("sigma_limit" instead of "sigma_decay") #3

Closed nlsfnr closed 2 years ago

nlsfnr commented 2 years ago

It seems that there is a typo in the SimpleGA strategy's "tell_strategy" method (line):

Instead of updating state["sigma"] with state["sigma"] * state["sigma_decay"] (correct), it says: state["sigma"] * state["sigma_limit"] (wrong).

RobertTLange commented 2 years ago

Good catch! Thank you very much. It is fixed in 57235ca15c75ec9804463f31286c1ff96b580708.