Skip to content

Change max_relative_growth unit to nan#1406

Merged
lkstrp merged 1 commit intomasterfrom
fix-unit-max-relative-growth
Oct 23, 2025
Merged

Change max_relative_growth unit to nan#1406
lkstrp merged 1 commit intomasterfrom
fix-unit-max-relative-growth

Conversation

@lisazeyen
Copy link
Copy Markdown
Contributor

@lisazeyen lisazeyen commented Oct 16, 2025 •
edited
Loading

Closes # (if applicable).

Changes proposed in this Pull Request

Correct unit for max_relativ_growth

Checklist

  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in docs.
  • Unit tests for new features were added (if applicable).
  • A note for the release notes docs/release-notes.md of the upcoming release is included.
  • I consent to the release of this PR's code under the MIT license.

@coroa
Copy link
Copy Markdown
Member

coroa commented Oct 16, 2025 •
edited
Loading

Why nan? Will this not break existing uses of only adjusting max_growth?

@lisazeyen
Copy link
Copy Markdown
Contributor Author

lisazeyen commented Oct 23, 2025

Why nan? Will this not break existing uses of only adjusting max_growth?

The nan is for the unit and not the default value. It is a relative factor without any unit which is getting multiplied with e.g. the installed capacity of the previous investment period

@lkstrp lkstrp changed the title Change max_relative_growth default value to nan Change max_relative_growth unit to nan Oct 23, 2025
@lkstrp lkstrp merged commit a69525b into master Oct 23, 2025
22 checks passed
@lkstrp lkstrp deleted the fix-unit-max-relative-growth branch October 23, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lkstrp lkstrp lkstrp approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lisazeyen @coroa @lkstrp