Skip to content

Commit

Permalink
Merge pull request #5411 from nabenabe0928/breaking/do-not-normalize-…
Browse files Browse the repository at this point in the history
…fanova

Do not normalize importance in f-ANOVA
  • Loading branch information
not522 committed Apr 25, 2024
2 parents 137b1b4 + 3f7d5c0 commit 1c19fd6
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions optuna/importance/_fanova/_evaluator.py
Original file line number Diff line number Diff line change
Expand Up @@ -126,8 +126,6 @@ def evaluate(
param_importances = np.array(
[evaluator.get_importance(i)[0] for i in range(len(non_single_distributions))]
)
# We normalize here to keep the backward compatibility.
param_importances /= np.sum(param_importances)

return _sort_dict_by_importance(
{
Expand Down

0 comments on commit 1c19fd6

Please sign in to comment.