morcibacsi / PSAVanCanBridge

VAN - CAN protocol bridge (V2C) for cars made by PSA Group (Peugeot, Citroen)
GNU General Public License v3.0
120 stars 27 forks source link

TRIP2 solution #24

Open droofy opened 3 years ago

droofy commented 3 years ago

This is a TRIP2 Work

morcibacsi commented 3 years ago

Have you checked it with DISPLAY_MODE set to 1 and 2 respectively? Is it working in both versions as expected?

https://github.com/morcibacsi/PSAVanCanBridge/blob/master/PSAVanCanBridge/Config.h#L31

droofy commented 3 years ago

Yes,

It’s work which both

Le 2 sept. 2021 à 10:10, Peter Pinter @.***> a écrit :

 Have you checked it with DISPLAY_MODE set to 1 and 2 respectively? Is it working in both versions as expected?

https://github.com/morcibacsi/PSAVanCanBridge/blob/master/PSAVanCanBridge/Config.h#L31

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

morcibacsi commented 3 years ago

So you mean, when the DISPLAY_MODE it is set to 2 then the "third page" on the display still shows the RPM at the first icon and the speed at the third icon?

droofy commented 3 years ago

DISPLAY_MODE it is set to 2 then the "third page" it displays the RPM well.

I did it differently, and it works for me.

So I leave the normal display : I created a DISPLAY_MODE 3 to test. I put the same thing as DISPLAY_MODE 1 by adding in "/PSAVanCanBridge/src/Can/CanDataSenderTask.h" :

        if (dataToBridge.LeftStickButtonPressed)
        {
            trip0Icon1Data = round(FUEL_TANK_CAPACITY_IN_LITERS * dataToBridge.FuelLevel / 100);
            trip0Icon3Data = dataToBridge.OilTemperature;
        }

Translated with www.DeepL.com/Translator (free version)

Le 2 sept. 2021 à 20:41, Peter Pinter @.***> a écrit :

DISPLAY_MODE it is set to 2 then the "third page"

thiagocanella commented 10 months ago

i will test this next week @droofy