search

signaflo / java-timeseries

Time series analysis in Java
MIT License
195 stars 49 forks source link

Consider replacing OffsetDateTime with library Time class #32

Closed signaflo closed 5 years ago

signaflo commented 5 years ago

OffsetDateTime is a critical but annoying piece of the library. I don't want external users of the library to have to mess with it, though there may be good reasons for keeping it around.

I would prefer to wrap it in a simple Time class that delegates the small chunk of behavior we need from it. If anything, we could get a much prettier toString representation and make it much easier to create an instance of the class.

signaflo commented 5 years ago

Replacement completed and merged into master.