Skip to content

Commit

Permalink
Fix bug in UE
Browse files Browse the repository at this point in the history
After the last update it was actually computing ucb
  • Loading branch information
ziatdinovmax committed Aug 20, 2023
1 parent f732710 commit 506c5ba
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions gpax/acquisition/acquisition.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@
import numpy as onp

from ..models.gp import ExactGP
from ..utils import random_sample_dict
from .base_acq import ei, ucb, poi
from .base_acq import ei, ucb, poi, ue
from .penalties import compute_penalty


Expand Down Expand Up @@ -321,7 +320,7 @@ def UE(rng_key: jnp.ndarray,
warnings.warn("`rng_key` is deprecated and will be removed in future versions. "
"It's no longer used.", DeprecationWarning, stacklevel=2)
return compute_acquisition(
model, X, ucb, noiseless,
model, X, ue, noiseless,
penalty=penalty, recent_points=recent_points,
grid_indices=grid_indices, penalty_factor=penalty_factor,
**kwargs)
Expand Down

0 comments on commit 506c5ba

Please sign in to comment.