Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix small issues in add_land_use_constraint_m #1022

Conversation

tgi-climact
Copy link
Contributor

Changes proposed in this Pull Request

Fix p_nom_min of renewables generators for myopic approach and add check of existing capacities in add_land_use_constraint_m. This reproduce the behaviour in add_land_use_constraint.

Checklist

  • I tested my contribution locally and it seems to work fine.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to envs/environment.yaml.
  • Changes in configuration options are added in all of config.default.yaml.
  • Changes in configuration options are also documented in doc/configtables/*.csv.
  • A release note doc/release_notes.rst is added.

…eck of existing capacities in add_land_use_constraint_m
@fneum fneum requested a review from lisazeyen April 14, 2024 18:16
@lisazeyen
Copy link
Contributor

@tgi-climact thank you!

@lisazeyen lisazeyen closed this Apr 16, 2024
@tgi-climact tgi-climact deleted the feature/fix-add-land-use-constraint-m branch May 21, 2024 07:09
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.

2 participants