[Ovmsdev] crash when calling 12 V history

[Michael Geddes]

https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/pull/892
This one is one me, and not sure how I missed it.

//.ichael

On Thu, 4 May 2023 at 15:45, Simon Ehlen <ovms at highteq.de> wrote:

> Hello Michael,
>
> after Michael Balzer's hint that I should take a closer look at the JSON
> string, I noticed that the error is in vehicle.cpp, which was changed in
> November.
>
> When concatting the string some elements (yAxis, comma, closing) were
> forgotten.
>
> You can see it if you compare for example the version of the Ioniq5
> against the one of the standard class.
>
> vehicle.cpp:
> https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/blob/master/vehicle/OVMS.V3/components/vehicle/vehicle.cpp#L2255
> hyundai_ioniq5:
> https://github.com/openvehicles/Open-Vehicle-Monitoring-System-3/blob/master/vehicle/OVMS.V3/components/vehicle_hyundai_ioniq5/src/hif_web.cpp#L347
>
> I would be grateful if you could create a PR, because I am currently not
> 100% sure whether in my vehicle.cpp not individual adjustments of me are
> in there, with which I would not like to "pollute" the official repo.
>
> What I noticed is that the gauges sometimes show different values than the
> label would lead you to believe.
>
> For example, instead of the engine temperature, the SOC is displayed.
>
> I am not familiar with the dashboard coding and would be grateful if you
> could show me where the values are assigned to the various gauges.
>
> Cheers,
>
> Simon
>
> Am 03.05.2023 um 02:10 schrieb Michael Geddes:
>
> It's probably worth seeing if you can see the fresult that is being
> returned causing the issue.. Being logged in the vfs-fat debug stream :/.
>
> Michael
>
> On Wed, 3 May 2023, 1:38 am Simon Ehlen, <ovms at highteq.de> wrote:
>
>> Hello,
>>
>> since a few days my OVMS crashes as soon as I call the 12 V history
>> monitor from the webinterface. The crash happens right after the click in
>> the menu even before the history web page is loaded.
>> I have not made any changes to my firmware recently nor any changes to
>> the 12 V history at all.
>> Yesterday I compiled the current version as a test, but the error occurs
>> there as well.
>>
>> Attached is the information from a2l with the backtrace.
>>
>> Cheers,
>> Simon_______________________________________________
>> OvmsDev mailing list
>> OvmsDev at lists.openvehicles.com
>> http://lists.openvehicles.com/mailman/listinfo/ovmsdev
>>
>
> _______________________________________________
> OvmsDev mailing listOvmsDev at lists.openvehicles.comhttp://lists.openvehicles.com/mailman/listinfo/ovmsdev
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openvehicles.com/pipermail/ovmsdev/attachments/20230504/cdce8290/attachment.htm>