-
Notifications
You must be signed in to change notification settings - Fork 438
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 of capex description #822
base: master
Are you sure you want to change the base?
Conversation
These CSV files fix the PyPSA issue PyPSA#817 with a corrected description for the capex attribute.
@Alex-Neve there seem to be some unrelated changes in the .csv files. Could you have a look? |
@FabianHofmann Good catch! I see what happened: I edited my files in Microsoft Excel and it automatically changed a few details (adding a dot after 0 values, turning "False" into "FALSE", adding quotes around the text strings...). I will commit new files directly edited in Notepad to avoid this sort of potential trouble. |
These CSV files fix the PyPSA issue PyPSA#817 with a corrected description for the capex attribute. These files versions are cleaned up from unrelated elements.
Fix description added for the next release.
This reverts commit 8b600df.
Fix of the capital_cost parameter description for all 6 attributes CSV files. The changes unrelated for this fix were avoided by using the latest version of the main branch of PyPSA.
I struggled a bit with some commits but the files look clean now. |
Closes PyPSA Issue #817.
Changes proposed in this Pull Request
Correction of the capex descriptions of the attributes CSV files.
Checklist
doc
.environment.yaml
,environment_docs.yaml
andsetup.py
(if applicable).doc/release_notes.rst
of the upcoming release is included.