-
Notifications
You must be signed in to change notification settings - Fork 4
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
Features/#392 include powerplants #453
Conversation
…om/openego/eGon-data into features/#392-include-powerplants
This feature got tested on the CI branch and the tasks were run successfully, but no data for carrier 'gas' and 'oil' git written to the database. This needs to be solved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I run this branch in testmode and it failed:
[2021-09-30 13:54:25,452] {logging_mixin.py:120} INFO - WARNING: No data returned by statement:
SELECT bnetza_id, name, carrier, capacity, postcode, city,
federal_state, c2035_capacity
FROM supply.egon_nep_2021_conventional_powerplants
WHERE carrier = 'oil'
AND chp = 'Nein'
AND c2035_chp = 'Nein'
AND c2035_capacity > 0
AND postcode != 'None';
[2021-09-30 13:54:26,083] {taskinstance.py:1150} ERROR - Cannot transform naive geometries. Please set a crs on the object first.
Traceback (most recent call last):
I think this happens because there are no oil power plants in Schleswig-Holstein.
I added an if...else and print statement to avoid problems with empty data frames imported from our NEP tables. |
There were some issues in the CHP extension, but after I merged the |
The task
The used table for non chp plants is filled in the CHP-data set, but this is not a dependency of the PowerPlant-Dataset. I think adding the dependency should solve the problem. |
Closes #392 by allocating and integrating conventional non CHP power plants into the power plants table.
Branch still needs to be tested for Germany on the CI branch