Skip to content

bugfixes in optimization#430

Merged
quaquel merged 1 commit intomasterfrom
objectives_optimization
Oct 29, 2025
Merged

bugfixes in optimization#430
quaquel merged 1 commit intomasterfrom
objectives_optimization

Conversation

@quaquel
Copy link
Copy Markdown
Owner

@quaquel quaquel commented Oct 29, 2025

This PR solves a few small bugs in optimization.

  1. It ensures that only outcomes that are of kind MINIMIZE or MAXIMIZE are used to create the platypus problem. This will prevent the situation of Tracking convergence not working when using Vensim model #279 from occurring.
  2. It removes expected_range from Outcome and its subclasses. This attribute is not used anymore since the changes in how convergence analysis is done.

@quaquel
removal of expected_range
9a9c4b8 
no longer relevant with the change of hypervolume calculations
@coveralls
Copy link
Copy Markdown

coveralls commented Oct 29, 2025

Coverage Status

coverage: 92.394% (+0.04%) from 92.359%
when pulling 9a9c4b8 on objectives_optimization
into c40d1d0 on master.

@quaquel quaquel linked an issue Oct 29, 2025 that may be closed by this pull request
Tracking convergence not working when using Vensim model #279
Closed
@quaquel quaquel merged commit ba7fbbf into master Oct 29, 2025
24 checks passed
@quaquel quaquel deleted the objectives_optimization branch October 29, 2025 19:17
@quaquel quaquel added the bug label Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Tracking convergence not working when using Vensim model

2 participants

@quaquel @coveralls