Time Type template for the ThreadsafeTimeseries - Githubissues

machines-in-motion / real_time_tools

BSD 3-Clause "New" or "Revised" License

9 stars 6 forks source link

Time Type template for the ThreadsafeTimeseries #11

Open MaximilienNaveau opened 5 years ago

MaximilienNaveau commented 5 years ago

Hi all,

It seems that the API would benefit to have a template type for the time tracking. write know it is a double, but people might want to use long int, chrono::XXX, ROS::Time, ...

I think this could be a 2 line improvement but that actually add a lot of flexibility in the final application where this is used.

wumanu commented 5 years ago

but then we would also have to change the way we measure the time and write it to the timestamp, no? not sure how this would be done.