Re: [Paparazzi-devel] BAT or ENERGY message improvement possible?

[Felix Ruess]

ok, this seems like a pretty harmless change...

Changed in master "centi-Amps". So you get a resolution of 0.01A in
the BAT message with a max of 327A which should still be enough...

Cheers, Felix

On Wed, Nov 2, 2011 at 1:09 PM, Christophe De Wagter <address@hidden> wrote:
> sounds good to me
> -Christophe
>
>
>
> On Tue, Nov 1, 2011 at 10:04 PM, Chris <address@hidden> wrote:
>>
>> Hi.
>> Is it possible if there is no higher cause to change the BAT message so it
>> can include
>> a more precise current value?
>> Up to know i had to include both "BAT" and "ENERGY" messages because of
>> the current field low resolution (The "BAT" message also includes the
>> throttle field so it is essential)
>> It looks like the "ENERGY" or "BAT" message can be modified in order to be
>> more complete/
>> Currently the value has a resolution of 1A but i changed it to:
>> #define PERIODIC_SEND_BAT(_chan) { \
>>    uint16_t amps = (int16_t) (current/100);                \ //was
>> (current/1000)
>>    Downlink({ int16_t e = energy;                    \
>>    DOWNLINK_SEND_BAT(_chan,                    \
>> &v_ctl_throttle_slewed,            \
>> &vsupply,                    \
>> &amps,                    \
>> &estimator_flight_time,            \
>> &kill_throttle,                \
>> &block_time,                    \
>> &stage_time,                    \
>> &e);                        \
>>      });                                \
>> }
>>
>> This way the current field has a resolution of 0.1A and still can display
>> more than enough amperes.
>> Chris
>>
>>
>> _______________________________________________
>> Paparazzi-devel mailing list
>> address@hidden
>> https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
>
>
> _______________________________________________
> Paparazzi-devel mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/paparazzi-devel
>
>