Skip to content

Feature/prevent simultanious flow rates#431

Open
FBumann wants to merge 10 commits intomainfrom
feature/prevent-simultanious-flow-rates
Open

Feature/prevent simultanious flow rates#431
FBumann wants to merge 10 commits intomainfrom
feature/prevent-simultanious-flow-rates

Conversation

@FBumann
Copy link
Member

@FBumann FBumann commented Oct 20, 2025

Description

Improve the flexibility of mutual exclusivity constraints in Components and add the option to the LinearConverter class

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Code refactoring

Related Issues

Closes #(issue number)

Testing

  • I have tested my changes
  • Existing tests still pass

Checklist

  • My code follows the project style
  • I have updated documentation if needed
  • I have added tests for new functionality (if applicable)

Summary by CodeRabbit

  • New Features

    • Components and converters can define multiple independent mutual‑exclusion constraint groups via prevent_simultaneous_flows (nested groups of flow labels).
  • Deprecated

    • Passing Flow objects to prevent_simultaneous_flows is deprecated; prefer flow label strings (object usage emits a deprecation warning).
  • Bug Fixes / Validation

    • Stronger validation and clearer errors for unknown or duplicate flow labels within groups.
  • Tests

    • Expanded unit and round‑trip serialization tests covering single/multiple groups, normalization, deprecation warnings, and error cases.
  • Documentation

    • Updated docstrings describing grouped constraints and deprecation.

Loading
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