Skip to content

Commit

Permalink
bugfix: make sure optim_sequential = False for aq_str in ['NEHVI', 'E…
Browse files Browse the repository at this point in the history
…HVI']
  • Loading branch information
xuyuting committed Aug 19, 2024
1 parent fcf8220 commit 18e634e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion obsidian/optimizer/bayesian.py
Original file line number Diff line number Diff line change
Expand Up @@ -774,7 +774,7 @@ def suggest(self,
aq_kwargs.update(self._parse_aq_kwargs(aq_str, aq_hps, m_batch, target_locs, X_t_pending, objective))

# Hypervolume aqs fail with X_t_pending when optim_sequential=True
if aq_i in ['NEHVI', 'EHVI']:
if aq_str in ['NEHVI', 'EHVI']:
optim_sequential = False

# If it's random search, no need to do optimization; Otherwise, initialize the aq function and optimize
Expand Down

0 comments on commit 18e634e

Please sign in to comment.