Skip to content

Declare gest-api as a core dependency - #306

Open
RemiLehe wants to merge 3 commits into
optimas-org:mainfrom
RemiLehe:add_gest_api_dependency
Open

RemiLehe wants to merge 3 commits into
optimas-org:mainfrom
RemiLehe:add_gest_api_dependency

Conversation

@RemiLehe

Copy link
Copy Markdown
Collaborator

gest-api is imported unconditionally by optimas.generators.base, optimas.diagnostics.exploration_diagnostics, and every bundled generator, but it was never declared in the project metadata. As a result, pip install optimas produces an installation where importing any generator raises ModuleNotFoundError.

CI masked this by installing gest-api explicitly after pip install .[test].

RemiLehe and others added 3 commits September 18, 2026 08:13
gest-api is imported unconditionally by optimas.generators.base,
optimas.diagnostics.exploration_diagnostics, and every bundled
generator, but it was never declared in the project metadata. As a
result, `pip install optimas` produces an installation where importing
any generator raises ModuleNotFoundError.

CI masked this by installing gest-api explicitly after `pip install
.[test]`.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
gest-api is now pulled in by `pip install .[test]`, so installing it
separately is no longer needed. Removing it also lets CI exercise the
declared project metadata rather than masking a missing dependency.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
All names optimas imports from gest_api (VOCS, ContinuousVariable,
DiscreteVariable, MinimizeObjective, ExploreObjective and Generator) are
already present in gest-api 0.1, so requiring 0.2 would needlessly
exclude a working version.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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