Skip to content
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

Checks for foreign network eGon100RE #404

Open
CarlosEpia opened this issue Feb 24, 2025 · 17 comments
Open

Checks for foreign network eGon100RE #404

CarlosEpia opened this issue Feb 24, 2025 · 17 comments
Assignees

Comments

@CarlosEpia
Copy link

CarlosEpia commented Feb 24, 2025

Here will be shared all the comments and checks regarding the foreign countries network. A comparison with the pypsa_eur results will be used. From the pypsa_eur results, only components that belong to these countries are shown: "AT", "CH", "CZ", "PL", "SE", "NO", "DK", "GB", "NL", "BE", "FR" and "LU".

@CarlosEpia CarlosEpia self-assigned this Feb 24, 2025
@CarlosEpia
Copy link
Author

Regarding buses we have:

Image

Shouldn't we have Li_ion buses?

@CarlosEpia
Copy link
Author

CarlosEpia commented Feb 24, 2025

Regarding links we have:

Image

  • Shouldn't we have BEV_charger?
  • I will take a look on the other missing links since some of them are aggregated or deleted intentionally:
    -- CH4 links between foreign countries are missing. @lenzim97 could be the case that they get deleted during the creation of CH4 or H2 grids in Germany?
    --

@CarlosEpia
Copy link
Author

Regarding generators we have:

Image

  • Only urban_decentral_solar_thermal is missing, but it is intentionally discarded in drop_urban_decentral_heat()

@CarlosEpia
Copy link
Author

Regarding storage_units:

Image

  • Everything looks OK.

@CarlosEpia
Copy link
Author

Regarding stores we have:

Image

  • Shouldn't we have battery_storage?

@CarlosEpia
Copy link
Author

Regarding loads we have:
Image

@CarlosEpia
Copy link
Author

From this point time-series data will be analysed. Since there are several tables for each kind of element, the comparison is presented in a table with multiindex: [table_name, carrier_name] and the values for the columns count-eGon100 and count-pypsa_eur correspond to how many columns of the same carrier there are in the table mentioned in the index.

Empty tables are ignored.

@CarlosEpia
Copy link
Author

Regarding link time-series:

Image

@CarlosEpia
Copy link
Author

CarlosEpia commented Feb 24, 2025

Regarding storage_units time-series:

Image

  • Looks OK

@CarlosEpia
Copy link
Author

Regarding stores time-series:

Image

  • Do we need e_min_pu values for battery_storage?

@CarlosEpia
Copy link
Author

Regarding load time-series:

Image

  • rural_heat and central_heat are supposed to be fixed in the next run
  • Land_transport_EV should be checked.

@ClaraBuettner
Copy link

Yes we should have all components modelling the mobility sector ("Li ion" buses, "land_transport_EV" loads, "BEV_charger" links, "battery_storage" stores) and all time series related to these.

I am quite sure that these have been there after the neigbour_reduction task, since I once aligned the carrier names.
So I guess it is again the problem that these components are deleted by another task.

@ClaraBuettner
Copy link

@CarlosEpia: I did not take a look at all the images but at your questions.

@CarlosEpia
Copy link
Author

Regarding the missing links when comparing eGon100RE vs pypsa_eur results:

<title></title>
<meta name="generator" content="LibreOffice 24.2.7.2 (Linux)"/>
<style type="text/css">
	body,div,table,thead,tbody,tfoot,tr,th,td,p { font-family:"Liberation Sans"; font-size:x-small }
	a.comment-indicator:hover + comment { background:#ffd; position:absolute; display:block; border:1px solid black; padding:0.5em;  } 
	a.comment-indicator { background:red; display:inline-block; border:1px solid black; width:0.5em; height:0.5em;  } 
	comment { display:none;  } 
</style>
Link eGon100RE
BEV_charger Missing
CH4 Missing
DAC Deleted intentionally
SMR_CC Deleted intentionally
central_gas_boiler Missing
gas_for_industry_CC Deleted intentionally
rural_air_heat_pump Included in rural_heat_pump
urban_central_gas_CHP_CC Included in central_gas_CHP
urban_decentral_air_heat_pump Included in central_heat_pump
urban_decentral_gas_boiler Deleted intentionally
urban_decentral_resistive_heater Deleted intentionally
urban_decentral_water_tanks_charger Deleted intentionally
urban_decentral_water_tanks_discharger Deleted intentionally

@ClaraBuettner do we agree that there is a problem related to the CH4 and central_gas_boiler links?

@ClaraBuettner
Copy link

When there are no central_gas_boilers and CH4 pipelines in foreign countries it is a problem, yes.

@CarlosEpia
Copy link
Author

The problem with the CH4 links is here:

# Clean db
db.execute_sql(
f"""DELETE FROM grid.egon_etrago_link
WHERE "carrier" = '{gas_carrier}'
AND scn_name = '{scn_name}';
"""
)

Both pipelines already executed that task. @ClaraBuettner and @lenzim97 does it make sense for you to stop one of the runs to include the changes regarding this problem and start it again?

@CarlosEpia
Copy link
Author

The problem with the central_gas_boilers links in foreign countries is here:

db.execute_sql(
f"""
DELETE FROM {targets['heat_links']['schema']}.
{targets['heat_links']['table']}
WHERE carrier LIKE '%central_gas_boiler%'
AND scn_name = '{scenario}'
"""
)

CarlosEpia added a commit that referenced this issue Feb 25, 2025
Fix missing CH4 and central_gas_boilers links in foreign countries
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

No branches or pull requests

2 participants