Merge pull request #208 from nabenabe0928/test/add-tests-for-ctpe

Add tests for c-TPE

package/samplers/ctpe/README.md

@@ -96,3 +96,15 @@ When you use this sampler, please cite the following:
     year={2022}
 }
 ```
+
+### Test
+
+To execute the tests for `cTPESampler`, please run the following commands. The test file is provided in the package.
+
+```sh
+pip install pytest
+```
+
+```python
+pytest package/samplers/ctpe/tests/
+```

package/samplers/ctpe/sampler.py

@@ -80,8 +80,8 @@ def __init__(
         )
 
     def _warning_multi_objective_for_ctpe(self, study: Study) -> None:
+        """TODO: Use this routine once c-TPE supports multi-objective optimization.
         if study._is_multi_objective():
-
             def _get_additional_msg() -> str:
                 beta = getattr(self._gamma, "_beta", None)
                 strategy = getattr(self._gamma, "_strategy", None)
@@ -98,6 +98,8 @@ def _get_additional_msg() -> str:
                 "but sampling will be performed by c-TPE based on Optuna MOTPE. "
                 f"{_get_additional_msg()}"
             )
+        """
+        self._raise_error_if_multi_objective(study)
 
     def _build_parzen_estimators_for_constraints_and_get_quantiles(
         self,
@@ -123,7 +125,7 @@ def _build_parzen_estimators_for_constraints_and_get_quantiles(
                     study, search_space, unsatisfied_trials, handle_below=False
                 )
             )
-            quantiles.append(len(satisfied_trials) / len(trials))
+            quantiles.append(len(satisfied_trials) / max(1, len(trials)))
 
         return mpes_below, mpes_above, quantiles
 
@@ -149,7 +151,7 @@ def _sample(
         mpes_above.append(
             self._build_parzen_estimator(study, search_space, above_trials, handle_below=False)
         )
-        quantiles.append(len(below_trials) / len(trials))
+        quantiles.append(len(below_trials) / max(1, len(trials)))
 
         _samples_below: dict[str, list[np.ndarray]] = {
             param_name: [] for param_name in search_space