cmaes icon indicating copy to clipboard operation
cmaes copied to clipboard

Scipy Dependency

Open nomuramasahir0 opened this issue 1 year ago • 4 comments

Discussion

Now I'm trying to introduce CMA-ES with Margin, and I'm thinking of putting scipy dependency into this repository. Specifically, I'd like to use chi2.ppf in the following. https://github.com/scipy/scipy/blob/v1.9.1/scipy/stats/_continuous_distns.py#L1343-L1407 It might be possible to implement chi2.ppf without scipy, but it seems to be a bit difficult (at least for me).

Optuna, which employs this repository for using CMA-ES, already requires scipy, so I think this doesn't become a problem. https://github.com/optuna/optuna/blob/master/setup.py#L39

nomuramasahir0 avatar Sep 01 '22 05:09 nomuramasahir0

What do you think? @c-bata

nomuramasahir0 avatar Sep 01 '22 05:09 nomuramasahir0

As for Optuna, we are currently considering removing scipy from the dependencies. However, I agree with the use of scipy in cmaes library :+1: Implementing CMA-ES with Margin is much more important than removing the scipy dependency.

c-bata avatar Sep 01 '22 06:09 c-bata

Ok, thanks! Btw, I can work for removing scipy in the next month (if you hope so). When we're implementing it from scratch, I think it is better to compare the accuracy and computational time with scipy.

nomuramasahir0 avatar Sep 01 '22 07:09 nomuramasahir0

I can work for removing scipy in the next month (if you hope so). When we're implementing it from scratch, I think it is better to compare the accuracy and computational time with scipy.

Sounds great 👍 As you said, we should compare the accuracy though.

c-bata avatar Sep 01 '22 07:09 c-bata

FYI: @amylase is working on this issue on https://github.com/CyberAgentAILab/cmaes/pull/126

c-bata avatar Nov 08 '22 01:11 c-bata