Skip to content

Solar aggregation in CCL plus minor fixes#1712

Merged
fneum merged 3 commits intoPyPSA:masterfrom
Aym-brz:master
Jun 20, 2025
Merged

Solar aggregation in CCL plus minor fixes#1712
fneum merged 3 commits intoPyPSA:masterfrom
Aym-brz:master

Conversation

@Aym-brz
Copy link
Copy Markdown
Contributor

@Aym-brz Aym-brz commented Jun 8, 2025

Changes proposed in this Pull Request

Improved the behavior of agg_p_nom_limits: added the ability to aggregate all solar electric technologies.
Improved the handling of heating capacities from before the first planning horizon: if no older heating capacity is available, a warning is displayed instead of causing an error later.

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.
  • Sources of newly added data are documented in doc/data_sources.rst.
  • A release note doc/release_notes.rst is added.

@JuliaSvn
Copy link
Copy Markdown
Contributor

JuliaSvn commented Jun 9, 2025

The solar aggregation add-on works fine on my side (no error) + the minor fixes solve some bugs, however the CCL constraints are still not working. Does any maintainer have an idea why it is not working ? When we print the constraint on p_nom they seem ok, however after the optimisation, the network does not respect the constraint anymore (which calls into question whether the model respects the other constraints?).

@Aym-brz Aym-brz changed the title Solar aggregation in CCL Solar aggregation in CCL plus minor fixes Jun 9, 2025
Copy link
Copy Markdown
Member

@fneum fneum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fneum fneum enabled auto-merge (squash) June 20, 2025 07:02
@fneum fneum merged commit 1e7c41f into PyPSA:master Jun 20, 2025
8 checks passed
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.

3 participants