[Rock-dev] Fwd: Re: Fwd: Re: IMUSensor - temperature and inclinometers values

[Jakob Schwendner]

On 09/26/2012 04:59 PM, Alexander Duda wrote:
> On 09/26/2012 04:24 PM, Matthias Goldhoorn wrote:
>> On 26.09.2012 16:22, Javier Hidalgo Carrió wrote:
>>> On 09/26/2012 03:33 PM, Matthias Goldhoorn wrote:
>>>> Hmh,
>>>>
>>>> i think we should add base::Temperature analog to base::angle 
>>>> WITHOUT an timestamp, becasue the temperature can then be added to 
>>>> any class without duplicate the time field.
>>>>
>>> In this case we will not be able to log the temperature alone in a 
>>> port with a timestamp. We will need to use the timestamp from other 
>>> port.
>> Thats the reason why you should add also the 
>> "base::TemperatureReading" type, which includes the time and the 
>> base::Temperature ;).
>>
>> In general in the past i was for an generic data type Stamped<T> 
>> which is in general for this kind of thinks, but there was some 
>> points against this, but i would like to point to it again ;).
>
> I do see Matthias point but I would rather go for:
> base::Temperature
> and
> struct base::samples::Temperature
> {
>     base::Time time;
>     base::Temperature temperature;
> }
>
> Alex
>
This seems to be most consistent with what we already have. Come to 
think of it I wouldn't even mind having an origin string field in the 
base::samples::Temperature type, as Matthias suggested.

Jakob


-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.dfki.de/pipermail/rock-dev/attachments/20120927/89f34c12/attachment.htm