Skip to content

[HOLD] Test default multinode objective weight#1063

Draft
mickaelbegon wants to merge 2 commits into
pyomeca:masterfrom
mickaelbegon:codex/fix-692-multinode-default-weight
Draft

[HOLD] Test default multinode objective weight#1063
mickaelbegon wants to merge 2 commits into
pyomeca:masterfrom
mickaelbegon:codex/fix-692-multinode-default-weight

Conversation

@mickaelbegon

@mickaelbegon mickaelbegon commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Context

The underlying behavior was already corrected on master when objective and constraint weights were separated: MultinodeObjectiveList.add() now creates an ObjectiveWeight() whose default value is 1. This PR locks that behavior to prevent a future regression to the historical zero-weight behavior.

Validation

  • MPLCONFIGDIR=/private/tmp/mplconfig-692 /Users/mickaelbegon/miniconda3/envs/bioptim/bin/python -m pytest tests/shard6/test_penalty.py::test_multinode_objective_default_weight_is_one -q
  • 1 passed

Closes #692


This change is Reviewable

@mickaelbegon mickaelbegon changed the title Test default multinode objective weight [HOLD] Test default multinode objective weight Jul 22, 2026
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.

penalty.weight=1 if not declared for multinode_objectives

1 participant