Skip to content

[v0.9.0] Fix: Rename solar panel KANENA to KANEKA - #536

Open
w-tusche wants to merge 3 commits into
PyPSA:masterfrom
w-tusche:fix/kaneka-solarpanel-name
Open

w-tusche wants to merge 3 commits into
PyPSA:masterfrom
w-tusche:fix/kaneka-solarpanel-name

Conversation

@w-tusche

Copy link
Copy Markdown
Contributor

Changes proposed in this Pull Request

Fix filename: A kaneka panel can be used, however, the filename and thus the name used to configure what is used is a type "kanena".

Fixed the file name and the 2 uses.

As this might lead to breaks when "KANENA" is configured it might be a good idea to add a future warning or just close the PR and don't merge it.

Checklist

  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in doc.
  • Unit tests for new features were added (if applicable).
  • Newly introduced dependencies are added to environment.yaml, environment_docs.yaml and setup.py (if applicable).
  • A note for the release notes doc/release_notes.rst of the upcoming release is included.
  • I consent to the release of this PR's code under the MIT license.

The panel config is from the manufacturer Kaneka,
the file name was a typo.
Panel names are derived from the filename,
so to use the Kaneka panel one had to use KANENA
@w-tusche w-tusche changed the title Rename solar panel KANENA to KANEKA Fix: Rename solar panel KANENA to KANEKA Oct 14, 2026
@euronion
euronion self-requested a review October 15, 2026 08:04
@euronion euronion added this to the release 0.9.0 milestone Oct 15, 2026
@euronion

Copy link
Copy Markdown
Collaborator

Thanks - we will fix it in a future release. I've added a warning to the release notes. For the next release.

@euronion euronion changed the title Fix: Rename solar panel KANENA to KANEKA [v0.9.0] Fix: Rename solar panel KANENA to KANEKA Oct 15, 2026
Comment thread RELEASE_NOTES.rst Outdated

This branch has not been deployed

No deployments
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