Open nikohansen opened 3 years ago
I believe the current cma
package can now deal with Fractions
. Hence the idea would be to see whether we still need the float
in the following assignment of the values that are later passed to tell
(line 500 in development@gitlab):
kernel._last_offspring_neg_UHVI_values = [-float(self.indicator_front.hypervolume_improvement(point))
The line https://github.com/CMA-ES/pycomocma/blob/036abf66f49bc339622562acefd1cef18e0fd3f6/comocma/como.py#L471 converts the HV computation result to
float
. This seems to defeat the very idea whyFractions
are available in themoarchiving
module.Possible solutions:
Fraction
type before the conversion. This could also be optional.cma
package work withFraction
and remove the conversion.