Skip to content

fix: test for configured carriers before adding land_use_constraint#139

Merged
tgilon merged 2 commits intomasterfrom
fix/test-carriers
Oct 15, 2025
Merged

fix: test for configured carriers before adding land_use_constraint#139
tgilon merged 2 commits intomasterfrom
fix/test-carriers

Conversation

@tgilon
Copy link
Copy Markdown
Member

@tgilon tgilon commented Oct 14, 2025

Closes #144.

Changes proposed in this Pull Request

Make sure that add_land_use_constraint uses the list of configuration renewable carriers defined in electricity:renewable_carriers.

Tasks

Workflow

Open issues

Notes

Checklist

  • I tested my contribution locally and it works as intended.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to envs/environment.yaml.
  • Changes in configuration options are added in config/config.default.yaml.
  • Changes in configuration options are documented in doc/configtables/*.csv.
  • Changes in configuration options are added in config/test/*.yaml.
  • Open-TYNDP SPDX license header added to all touched files.
  • Sources of newly added data are documented in doc/data_sources.rst.
  • New rules are documented in the appropriate doc/*.rst files.
  • A release note doc/release_notes.rst is added.
  • Major features are documented with up-to-date information in README and doc/index.rst.

@tgilon tgilon requested a review from daniel-rdt October 14, 2025 12:44
@tgilon tgilon self-assigned this Oct 14, 2025
@tgilon tgilon added this to the Release v0.3 milestone Oct 15, 2025
@tgilon tgilon added bug Something isn't working S SB Scenario Building labels Oct 15, 2025
@tgilon tgilon merged commit 52e07ab into master Oct 15, 2025
5 of 7 checks passed
@tgilon tgilon deleted the fix/test-carriers branch October 15, 2025 13:22
@tgilon tgilon mentioned this pull request Oct 16, 2025
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working S SB Scenario Building

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TYNDP trajectories overwritten for onwind

2 participants