AnimatedPlotterly doesn't accept a mix of offset-naive and offset-aware datetimes

dstl / Stone-Soup

A software project to provide the target tracking community with a framework for the development and testing of tracking algorithms.

https://stonesoup.rtfd.io

MIT License

384 stars 126 forks source link

AnimatedPlotterly doesn't accept a mix of offset-naive and offset-aware datetimes #908

Closed orosoman-dstl closed 6 months ago

orosoman-dstl commented 6 months ago

Title. Doing this will lead to a TypeError when Stone Soup tries to do arithmetic using the datetimes. This can be fixed in a couple of lines by converting using the tzinfo property, maybe giving a warning if that's necessary?

sdhiscocks commented 6 months ago

So this issue will be caused with the input data. If all data that is read into Stone Soup objects is TZ aware, this shouldn't be an issue. Having mix of both will cause issues in a number of places, not just with plotter.

orosoman-dstl commented 6 months ago

OK, fair enough.