search

BESTenergytrade / simply

Simulation of Electricity Markets in Python
MIT License
4 stars 1 forks source link

Actor creates orders with Nan energy #193

Closed j-ti closed 1 year ago

j-ti commented 1 year ago

In case actor reaches end of time series, it will create an order with an energy and order type of np.nan. Instead order should not be generated (a warning could be raised if actually data is missing)

PaulScheerRLI commented 1 year ago

I think an exception should be thrown, since not just order generation will not be possible but also handling of battery/schedule since most like the schedule is nan as well.

j-ti commented 1 year ago

I also think that an exception can be thrown, but maybe it is helpful if additionally a warning is raised within the actor already when an order is generated with a NaN order type and energy, so the source of error is more clear.