You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
"Force parking every restart" function of the stepper motor idle runs every MCU restart. This is only useful if MCU is powered from car first. Since we power MCU from usb before car is started, this effectively skips init procedure, result in inconsistent starts due to idle valve being whatever it was.
Hardware
Board: disco
Engine info: anything with a stepper idle valve
To Reproduce
Steps to reproduce the behavior:
Plug USB into controller - 12v not present to power actuators
Watch idle valve not move
Wait sufficient amount of time for init procedure to be over ((steps +extra steps) * reaction time)
Provide 12v ignition
Watch stepper not move
Expected behavior
Stepper Idle valve to be parked when it can actually move
Screenshots
Additional context
There should be an option: "delay stepper park until Vbat > X level" where X is either hardcoded at lets say 7v or user-settable.
The text was updated successfully, but these errors were encountered:
ggurov
changed the title
Stepper motor's reset function runs on MCU boot, not when battery voltage is present.
Stepper motor's reset function runs on MCU boot, not when battery voltage is present and can actually move the stepper
May 3, 2024
additionally, stepper valve should re-init position and time Vbat goes from <5v to that X level above, either 7v or user-settable as there could be missed steps, and such loss of Vbat would indicate key off -> key on.
Describe the bug
"Force parking every restart" function of the stepper motor idle runs every MCU restart. This is only useful if MCU is powered from car first. Since we power MCU from usb before car is started, this effectively skips init procedure, result in inconsistent starts due to idle valve being whatever it was.
Hardware
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Stepper Idle valve to be parked when it can actually move
Screenshots
![image](https://private-user-images.githubusercontent.com/2112173/327762141-6b250bb9-8974-4291-bb36-2e8abb345d77.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQ1MDc3OTIsIm5iZiI6MTcyNDUwNzQ5MiwicGF0aCI6Ii8yMTEyMTczLzMyNzc2MjE0MS02YjI1MGJiOS04OTc0LTQyOTEtYmIzNi0yZThhYmIzNDVkNzcucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDgyNCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA4MjRUMTM1MTMyWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9YTEzMTJjNTljMjNlN2E3OGQ0OGYyNWYxZTg0N2I4ZTQ1M2QwNzcyNTNlZjEzM2Y5MWQ5NDM5NDc4ZDBlZmNjMiZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.42hXHBw7R-FZk48TzRr3LYgvYfn4E5sqjEpOTsoX_O4)
Additional context
There should be an option: "delay stepper park until Vbat > X level" where X is either hardcoded at lets say 7v or user-settable.
The text was updated successfully, but these errors were encountered: