-
Notifications
You must be signed in to change notification settings - Fork 12
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
Example on stratified thermal storage might be confusing regarding used parameters #129
Comments
I would like to use this Issue to collect some suggestions for further improvements on the examples.
|
I would open a PR and realize the proposed improvements, if you could give me a brief feedback on the points I mentioned above, @jnnr and @c-moeller. |
I will introduce the changes in PR #135. |
In the docu it says:
|
Related to my earlier question...
What do I have to select if I want to expand the |
I'm not sure, if this answers your questions (and even not sure, if I am completely right), but here are some thoughts:
|
Ok, thaks for clearifying this.
|
The points @c-moeller makes are correct. In case of expandable, capacity is the existing capacity. This is true for all oemof.tabular facades. If this is not well explained in the docs, we can expand it. |
Well, if that is the case the docu is correct and clear. |
@jnnr, could you give me a feedback on the questions/points I stated at the very beginning of this issue, please. |
You are right. The facade has only a single parameter 'efficiency'. If the user does not use the facade (as shown in stratified_thermal_storage.py, stratified_thermal_storage_investment_option_1.py, stratified_thermal_storage_investment_option_2.py), she has to pass inflow_conversion_factor and outflow_conversion_factor, which have the same value in the examples. To make it simpler, I propose that in the .csv we keep a single parameter, efficiency, and use it in the examples with and without facade. |
I agree. Let's drop the beginning 'stratified_thermal_storage_'.
Proposal for renaming: Maybe these are too many examples? We could even drop some of the examples that do not use the facade.
Good idea to describe a bit more in the header. You could take the text from the docs which explains the difference.
Yes. Also, to make stratified_thermal_storage_facade.py work you have to first execute stratified_thermal_storage.py. We could drop the part comparing the lp-files. Or merge it into one example? |
@jnnr: Thanks for your feedback!
|
I will withdraw my self-assignment because I won't have the time to work on this any more. |
@jnnr is this now fixed with you latest work? Can we close this issue? |
The major part of it is addressed. |
As far as I understand the stratified thermal storage facade has the parameter
efficiency
. In the examples I found thatstratified_thermal_storage.csv
holds the parametersinflow_conversion_factor
andoutflow_conversion factor
. To me that is confusing because I understand all of them to have the same effect (apart from: the later differenciates between input and output).Is that on purpose? Is there a use to have all three of them?
The text was updated successfully, but these errors were encountered: