GenericStorage not creating InvestmentFlows when used with invest_relation_input_output about oemof-solph HOT 5 OPEN

Shouldn't the Flows be created with nominal_value=Investment() to allow for investments in any case?

from oemof-solph.

That might work (have to try it tomorrow) but it isn't shown like that in any of the examples. And then the _set_flows function wouldn't make much sense to me? Or is there another reason why the Investment objects in the flow should be overwritten? Maybe because of the lifetime_inflow`lifetime_outflow` attributes?

from oemof-solph.

Maybe, it makes sense to rewrite this thing anyway. I'd consider replacing the fixed relations between the investments by a limit to the c-rate. It would be an internal property that exists independent from the nominal values of the flows.

from oemof-solph.

Yeah, you are right about locating this to the _set_flows method, @lensum , I think.

• The case of invest_relation_input_output is not accounted for here. In my view, a check needs to be added in _set_flows.
• Also, for the other cases, the type of flow.investment is evaluated. I think, this needs to be flow.nominal_value instead according to the latest structural changes.

from oemof-solph.

I see multiple problems with the current implementation. First of all, we create unexpected investment variables. (If the Flows do not have to have one, they are silently created.) Secondly, you can do confusing things. The invest_relation_input_capacity=0.1 will be typically read as "it needs 10 hours to charge the storage". However, you can set a "max" to the flows to bypass that:

electric_storage = cmp.GenericStorage(
    label="electric_storage",
    inputs={el_bus: solph.Flow(max=10)},  # will be charged in 1 hour
    outputs={el_bus: solph.Flow()},
    nominal_storage_capacity=solph.Investment(
        ep_costs=elstor_epc, existing=1200, maximum=2100
    ),
    invest_relation_input_capacity=0.1,
    invest_relation_input_output=0.1,
)

from oemof-solph.