Skip to content

Vectra C Climate menu issue #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
PNKP237 opened this issue Apr 7, 2025 · 8 comments
Open

Vectra C Climate menu issue #7

PNKP237 opened this issue Apr 7, 2025 · 8 comments

Comments

@PNKP237
Copy link
Owner

PNKP237 commented Apr 7, 2025

I found a problem with the climate control. When the device is working and there is a connection via the CAN bus, then after entering the climate control menu, the navigation through the menu and selection of parameters occurs spontaneously. I pressed and held 9 (restarting the device) - the problem went away. As soon as the inscription "ehu32..." appeared on the screen, the problem appeared again.

https://youtube.com/shorts/Iss1vI-4Hbk?si=M63rFR_Zk_dIc0jw

Originally posted by @Technolog83 in #1

The problem only went away for a while because EHU32 was in the "CAN learning process", testing various IDs.
Simulation of button presses in climate menu is a feature for quickly enabling or disabling AC and that works as intended in Astra and Zafira. If you hold the knob button pressed for half a second, then the simulation should NOT occur.
EHU32 does not know what climate screen is displayed, it just presses the buttons to switch the AC on or off.

Seems like some inputs are being dropped on your vectra, I'll try to extend the time between simulated presses and give you a new version to test later today.

@Technolog83
Copy link

Technolog83 commented Apr 7, 2025

Ok!
If I make a short press on the climate button - it turns on the AC. If I make a long press on the climate button - it goes into the menu. But if I go into the menu and make a choice of the flow direction, then I make a choice by short pressing and enu32 again make the emulation of turning on the climate. Turning on the climate should be a long press of the button, about 2 seconds.

@PNKP237
Copy link
Owner Author

PNKP237 commented Apr 7, 2025

I see what you mean. It is working as intended then, the long press will not enable the AC automatically so the user can adjust whatever blower settings they want.
I like the quick press toggle though, but you gave me an idea - I'll extend the time before is starts simulating presses, so that if nothing happens for lets say 3 seconds it will toggle the AC, but won't do anything if it detects the user rotating or pressing on the AC panel.
Same if the user starts doing something, any action by the user will stop the automatic AC toggle.

It will take few days to complete though so I can't give you anything today.

@Technolog83
Copy link

Technolog83 commented Apr 7, 2025

If I press the control and do not press anything else for 3 seconds, the air conditioner will turn on - this is good. But if I press the control, go to the air flow selection menu, and then get distracted for 4-5 seconds (after all, I am driving a car), then the EHU32 will simulate turning on the air conditioner, but will not turn it on, because I will be in the air flow selection menu.

The EHU32 cannot track when I press the climate button - in which menu I am (the main climate screen or the air flow direction selection menu), so the simulation of turning on the climate may be incorrect.
Or the second situation: I entered the climate menu, changed something in the settings and remained in the menu, EHU32 can simulate turning on the AC - this will not be correct.

I think that a more reliable solution would be to turn on the AC by long pressing the climate button - 2 seconds. If this can be implemented in the firmware. Or make it possible to disable this function.

@PNKP237
Copy link
Owner Author

PNKP237 commented Apr 7, 2025

But if I press the control, go to the air flow selection menu, and then get distracted for 4-5 seconds (after all, I am driving a car), then the EHU32 will simulate turning on the air conditioner, but will not turn it on, because I will be in the air flow selection menu.

it will not, because the simulation would already be stopped if you pressed anything else after the first press.
I think there is a way to track which position in the menu is selected, I noticed this with UHP menu on my desktop setup. I'll explore how viable is that later this week.

Of course, being able to configure this live in vehicle would be the best but it's difficult to implement things like menus with the way these displays operate

@PNKP237
Copy link
Owner Author

PNKP237 commented Apr 9, 2025

I put it to 1 second for now. If you want to enter that menu normally you likely press it for less than 0,3s anyway, myself I honestly never use the menu since I just rotate the knob until I get the vent setting I want. Additionally found a bug in astra/zafira ECC but a fix will wait until v0.9.5 is ready.
Also implemented that if any movement/press is detected while the simulation is running, the simulated presses will stop immediately and ignore any inputs for the next 10 seconds.
You can test it if you have some spare time.

EHU32_094_VCACFIX.zip

@Technolog83
Copy link

I checked the latest firmware version. It turned on the AC once and never again. It doesn't respond to long pressing of the climate control button.

@PNKP237
Copy link
Owner Author

PNKP237 commented May 8, 2025

I'm about to release v0.9.5. For now, the AC toggle function will only trigger if the button has been held down for at least ~0,4s.
Short presses don't trigger the script.
This took so long because of the AC button in Astra ECC not following the GM standard (only reports button press and release, so I have to count the time on my own).

Additionally fixed CD70/DVD90 sometimes displaying "Aux", now it will never show up. I would be grateful if you could test it in your Vectra when you have a chance.

EDIT: My bad, I sent the wrong version (0.9.5d pre02) which had no AC fix yet. pre03 is fixed

EHU32_v0.9.5d_pre03.zip

@Technolog83
Copy link

I am currently using version 0.9.4, no problems for 2 weeks! Even the AC works! I don't understand why.
I will test version 0.9.5 and report the result.

Я збираюся випустити версію 0.9.5. Наразі функція перемикання кондиціонера спрацьовуватиме лише тоді, коли кнопка утримується щонайменше ~0,4 секунди. Короткі натискання не запускають скрипт. Це зайняло так багато часу через те, що кнопка кондиціонера в Astra ECC не відповідає стандарту GM (повідомляє лише про натискання та відпускання кнопки, тому мені доводиться рахувати час самостійно).

Додатково виправлено помилку, через яку CD70/DVD90 іноді відображав "Aux", тепер він ніколи не відображається. Буду вдячний, якщо ви зможете перевірити це у своїй Vectra, коли матимете таку можливість.

РЕДАГУВАННЯ: Моя помилка, я надіслав неправильну версію (0.9.5d pre02), у якій ще не було виправлення AC. pre03 виправлено.

EHU32_v0.9.5d_pre03.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants