Skip to content

Update Egor wrapper to egobox 0.38 - #10

Open
relf wants to merge 2 commits into
LSDOlab:mainfrom
relf:update-egor-wrapper
Open

relf wants to merge 2 commits into
LSDOlab:mainfrom
relf:update-egor-wrapper

Conversation

@relf

@relf relf commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Overview

Ports the Egor wrapper to the egobox 0.38 Python API.

Breaking

  • Requires egobox>=0.38.
  • The constraint columns of results["y_doe"] now hold the raw modOpt constraint values, not egobox's internal c(x) <= 0 form (an egobox 0.38 change).

Changes

  • Deprecated options (still work, with a DeprecationWarning): n_start → infill_n_start, doe → x_doe/y_doe, cstr_tol → cstr_tols.
  • New options: feasible_infill_strategy, stop_on_error. Combining EFI with LOG_EI raises a ValueError at setup.
  • Moved to minimize(): outdir, warm_start, hot_start and verbose.
  • Tolerances: cstr_tols takes a scalar or one value per bounded constraint, set on each CstrSpec. Fixes wrong tolerance sizing for equality and double-sided constraints.
  • Defaults: gp_config, qei_config, infill_n_start and target default to egobox's own defaults.
  • Results: success is False when egobox exits on an error.
  • Evaluation: the objective and constraints are evaluated point by point, so drivers that cache by point (e.g. OpenMDAO) return matching values.

Changes (Check all boxes that apply)

  • Added tests
  • Added examples
  • Updated docs
  • Refactor code
  • Added functionality
  • Fixed bugs (Patch version increase)
  • Changed API (Major/Minor version increase)
  • New release (Minor version increase): 0.4.0

Checklist

  • All existing tests pass
  • No local merge conflicts
  • Code is commented
  • Tests written for new features (if needed)
  • Version number is updated (if needed)

Optional

Related Github Issues

  • Resolves #

New Dependencies

None

@relf
relf marked this pull request as ready for review October 1, 2026 17:35

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant