Ovi Tătar

Expose current time in `Broker`

3

I have another small suggested addition. This PR is less important to me than the other PRs made this week. Instead of having to assume the current time (which I...

Add type to `Broker`'s `_actor_infos`

2

This is one of the solutions I suggested - fixes #203. I'm not sure it is the solution that is the best, because it changes the interface of the broker...

`SimpleBattery` is sensitive to floating point imprecision

1

I was using randomly assigned values for initializing a `SimpleBattery`, and I noticed the `update` method sporadically failed at this line: https://github.com/dos-group/vessim/blob/e7fe48ecbf68c3c6b05e437ef96090fa7db8189c/vessim/storage.py#L57 This is despite the initialization being valid, and...

Working with the new interface, I now notice a shortcoming of removing the actors from the pickled Microgrid representation: From the broker, it is practically impossible to know which actor...

Resampling in `HistoricalSignal.forecast` off by 1 if outside available forecast data

2

I'm back again with some more (from a user perspective) unexpected behaviour :relaxed: Take any `HistoricalSignal` that includes separate forecast information, e.g. one of the solcast datasets. ```python3 >>> start...