-
Notifications
You must be signed in to change notification settings - Fork 11
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
Sankey should include solar and wind with battery #4240
Comments
Thanks for this @MartLubben, well-spotted! |
I have examined the issue. I would expect that these energy flows would be implemented in the conversion_from_solar_electricity_to_electricity & conversion_from_wind_electricity_to_electricity queries, both the nodes for solar (energy_battery_solar_electricity) and wind (energy_battery_wind_electrcity) with batteries fall under the electricity_production group. However, these queries filter based on solar_radiation_input_conversion and wind_input_conversion respectively. I see 2 options to fix this issue:
@mabijkerk Do you agree? And which option would have your preference? |
I think the reason why the As becomes clear through this issue, this approach has its downsides. Another downside is that when we write queries using I think this is a relevant issue for the implementation of hybrid offshore wind as well. I suggest we discuss this issue in a short meeting for that project. |
"Zon op land met batterijsysteem" and "Op land met batterijsysteem" are not included in the Sankey.
The text was updated successfully, but these errors were encountered: