Feature/aggregate rework v2+status

[FBumann]

Description

Improve the handling of Status in clustered Models

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

  • Added cluster_mode parameter with 'relaxed' and 'cyclic' options for clustered system modeling.

• Enhancements

  • Improved startup/shutdown handling with relaxed mode for systems without previous status data.
  • Added cyclic constraint enforcement to maintain consistent system status at cycle boundaries.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

📝 Walkthrough

Walkthrough

The changes introduce a new cluster_mode parameter to StatusParameters and InvestParameters with values 'relaxed' or 'cyclic', enabling relaxed mode where previous state and duration values can be None. Modeling logic now conditionally applies constraints based on this mode, and a cyclic constraint enforcement method is introduced for clustered systems.

Changes

Cohort / File(s)	Summary
Cluster Mode Parameter Addition flixopt/interface.py	Added cluster_mode: Literal['relaxed', 'cyclic'] = 'relaxed' parameter to StatusParameters and InvestParameters constructors; updated typing imports to include Literal.
Relaxed Mode Support in Modeling Primitives flixopt/modeling.py	Updated consecutive_duration_tracking() to accept previous_duration: None, with initial duration constraint now conditional on non-None value. Updated BoundingPatterns.state_transition_bounds() to accept previous_state: None, returning optional initial constraint (None in relaxed mode). Big-M computation treats None previous values as 0.
Constraint Application and Cyclic Logic flixopt/features.py	Introduced _add_cluster_cyclic_constraint() to enforce equality of status at time 0 and final step for cyclic mode. Modified startup/shutdown handling to compute previous_state from _previous_status with None fallback. Updated _get_previous_uptime() and _get_previous_downtime() to return None in relaxed mode instead of default values.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Possibly related PRs

• Feature/rename on off to status #500 — Modifies the same StatusParameters/StatusModel surface and updates shared modeling functions (consecutive_duration_tracking and BoundingPatterns.state_transition_bounds).

Suggested labels

v3.0.0

Poem

🐰 A cluster hops in two new ways,
Relaxed or cyclic through the days,
With states that bloom and durations dance,
Constraints now flow with gentle stance! 🌀

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description is incomplete; it only provides a brief one-line summary ('Improve the handling of Status in clustered Models') and leaves all template sections (Type of Change, Related Issues, Testing) largely unfilled or unchecked.	Complete the description by selecting the appropriate Type of Change (appears to be New feature), specifying the issue number, and confirming whether testing was performed and tests still pass.
Docstring Coverage	⚠️ Warning	Docstring coverage is 77.78% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Title check	❓ Inconclusive	The title is vague and does not clearly convey the main change; 'Feature/aggregate rework v2+status' uses non-descriptive terms and branch naming conventions rather than explaining what was improved.	Revise the title to clearly describe the feature, e.g., 'Add cluster mode support for status modeling in aggregated systems' or 'Support relaxed and cyclic status modes for clustered models'.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[FBumann]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.