[Ovmsdev] crash when calling 12 V history

[Michael Geddes]

Yeah. The order is critical.

I should probably have done it a slightly different way for the final
putting together.

Michael

On Thu, 4 May 2023, 4:24 pm Michael Geddes, <frog at bunyip.wheelycreek.net>
wrote:

> 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/4e05ca59/attachment.htm>