vessim icon indicating copy to clipboard operation
vessim copied to clipboard

Expose current time in `Broker`

Open Impelon opened this issue 1 year ago • 3 comments

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 do now) or extracting it as the last value from one of the time series, it could be helpful to export this property, too.

Another (only indirectly related) aspect one may want to think about converting these times (or their type annotations) to a common representation, too. (Some kind of datetime object instead of a DatetimeLike.)

Impelon avatar May 03 '24 18:05 Impelon

Right, I didn't notice the merge conflicts, I fixed them now :)

Impelon avatar May 22 '24 10:05 Impelon

We're currently discussing a redesign of the actor interface and are putting development of the SiL module and also the Broker as this will probably influence their interface as well.

marvin-steinke avatar May 22 '24 10:05 marvin-steinke

I will check for compatibility with our new stuff and merge in the next couple of days :)

marvin-steinke avatar Jul 22 '24 11:07 marvin-steinke