Skip to content

Commit

Permalink
Merge: Correct typo in docs on acquisition functions (#442)
Browse files Browse the repository at this point in the history
There was a small typo.
  • Loading branch information
Scienfitz authored Dec 13, 2024
2 parents 53ce10e + 91e3e92 commit 813ab62
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Search spaces are now stateless, preventing unintended side effects that could lead to
incorrect candidate sets when reused in different optimization contexts
- `qNIPV` not working with single `MIN` targets
- Passing a `TargetTransformation` without passing `bounds` when createing a
- Passing a `TargetTransformation` without passing `bounds` when creating a
`NumericalTarget` now raises an error
- Crash when using `ContinuousCardinalityConstraint` caused by an unintended interplay
between constraints and dropped parameters yielding empty parameter sets
Expand Down
4 changes: 2 additions & 2 deletions examples/Basics/recommenders.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,9 +84,9 @@
# - If a larger value is chosen, only those that allow batching.
# That is, 'q'-variants of the acquisition functions must be chosen.

# The default he acquisition function is q-Expected Improvement.
# The default acquisition function is q-Log Expected Improvement.

ACQ_FUNCTION = "qEI"
ACQ_FUNCTION = "qLogEI"

### Other parameters

Expand Down

0 comments on commit 813ab62

Please sign in to comment.