-
Notifications
You must be signed in to change notification settings - Fork 13
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
import and connect generators #21
Comments
New geno mviews are considered in dingo by now, see openego/ding0#225. |
Note: conv. genos are disregarded now, due to a bug in conv pp tables, see openego/data_processing#175 |
Notes:
|
When there is one aggr. Generation, neglecting its type, I don't know how to assign the according time series... Any suggestions for this? |
Ok I will change this.. |
A question concerning cable selection procedure @birgits @gplssm : To connect new genos I have to choose a cable type for line. Approach: Use the smallest possible cable available which is able to carry the max. apparent power (taking the load factor into account). If the power exceeds the max. available value, use the quantity attribute (same procedure here -> In your parts, you have to select cables too - is there a generic function to do this? If not, I'll move mine to e.g. tools... |
For grid expansion there are only two possibilities. Either a parallel cable of the same kind is installed or if this isn't sufficient as many parallel standard cables as needed are installed. So to answer your question there is no function that chooses the smallest possible cable yet, but you did the same in dingo, right? Your equation is correct, though for lines we use current instead of power. Otherwise you'd have to know the line voltage to calculate S. |
Thanks for the hint! EDIT: Solved, new method. Cf. #45 (comment) |
@nesnoj should we keep this issue open as a reminder what's left to do here or are just conventional generators missing, in which case I would just make a new issue out of it? Is the ToDo list at the top up to date? |
Is this one still current? |
It is still relevant, mostly because conventional generators are not yet imported. |
Exactly my favorite kind of humor 😁
The feature of determining dispatch of flexible power plants is planned to be included when OPF is implemented in eDisGo. But, we can remove it from the list respectively mark it as resolved as we have the option to provide data that determines the dispatch.
This seems to me still important. I would propose to create a new issue for that.
Important. Still open. Issue exits. I deleted it. Some of the tasks that are described above in the issue's description, I didn't understand. @nesnoj are this task still relevant and open?
I suggest to create new issues. @birgits said that some flexible power plant technologies (i.e. biomass, CHP,...) aren't imported at the moment. If this is true, let's created an issue. Info: Some other points were removed due to risk of pending forever. |
I am not totally sure as I don't fully understand what is done but in those lines a filter for renewable generators is set so that only solar and wind generators are retrieved and the import of conventional generators is commented. The reason why this is done is not mentioned. I guess the problem with conventional generators could be the changing IDs that are addressed in issue #59. But I can't think of a reason why e.g. biomass powerplants can't be imported. |
Filters are now removed and all types of generators imported. The issue can now be closed. |
Implement import of generator, weather and timeseries datasets.
Tasks:
Don't forget to remove version-workaround in model draft import
(Move pre-data_processing of RES and Conv Power plants to DP? data_processing#180) (including col
preversion
) !la_id
to geno id, solution see Distinguish aggr. generation by type not by subtype #51 (comment)(to be solved first: decommissioning of conv. PP data_processing#178 and changing ids of conv generators in different scenarios data_processing#175)
[ ] make functions more generic (for use with data from pyPSA), esp._update_grids
[ ] Weather data[ ] Import[ ] ...[ ] Log what happenedThe text was updated successfully, but these errors were encountered: