[Rock-dev] Mistake in TimestampEstimator documentation?

[Leif Christensen]

Hi,

rereading the TimestampEstimator documentation
(http://www.rock-robotics.org/master/documentation/data_processing/timestamp_estimator_usage.html),
I stumbled upon the following:

> As an example, the setup of the Xsens IMU, when configured at 10Hz, would be:
> 
> mTimestamper = aggregator::TimestampEstimator(
>     base::Time::fromSeconds(10),
>     base::Time::fromMilliseconds(10));

Second argument is the device period, so for a 10Hz device this should be

> mTimestamper = aggregator::TimestampEstimator(
>     base::Time::fromSeconds(10),
>     base::Time::fromMilliseconds(100));

right?

LG,
Leif

---
 Leif Christensen

 DFKI Bremen
 Robotics Innovation Center
 Robert-Hooke-Straße 5
 28359 Bremen, Germany

 Phone: +49 (0)421 17845-4149
 Fax:   +49 (0)421 17845-4150
 E-Mail: leif.christensen at dfki.de

 Weitere Informationen: http://www.dfki.de/robotik
 -----------------------------------------------------------------------
 Deutsches Forschungszentrum fuer Kuenstliche Intelligenz GmbH
 Firmensitz: Trippstadter Straße 122, D-67663 Kaiserslautern
 Geschaeftsfuehrung: Prof. Dr. Dr. h.c. mult. Wolfgang Wahlster
 (Vorsitzender) Dr. Walter Olthoff
 Vorsitzender des Aufsichtsrats: Prof. Dr. h.c. Hans A. Aukes
 Amtsgericht Kaiserslautern, HRB 2313
 Sitz der Gesellschaft: Kaiserslautern (HRB 2313)
 USt-Id.Nr.:    DE 148646973
 Steuernummer:  19/673/0060/3
 -----------------------------------------------------------------------