Skip to content
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

cannot calibrate ETB/TPS without accel pedal #433

Open
nmschulte opened this issue May 25, 2024 · 2 comments
Open

cannot calibrate ETB/TPS without accel pedal #433

nmschulte opened this issue May 25, 2024 · 2 comments

Comments

@nmschulte
Copy link
Contributor

nmschulte commented May 25, 2024

EtbController won't return successful init unless there's an accel pedal present: https://github.com/FOME-Tech/fome-fw/blob/master/firmware/controllers/actuators/electronic_throttle.cpp#L165

This seems unnecessary, for both TPS auto-calibration and PID autotune.

Perhaps init should remove this check? Unsure how ETB pedal target (m_pedalMap) would behave if no channels configured.

@mck1117
Copy link
Collaborator

mck1117 commented Jan 1, 2025

That seems like a reasonable restriction, why would somebody run an ETB without a pedal?

@nmschulte
Copy link
Contributor Author

mck1117 commented

why would somebody run an ETB without a pedal?

nmschulte commented

This seems unnecessary, for both TPS auto-calibration and PID autotune.


At the time, I had no need for a pedal -- bench testing/development I suppose.

It's no big deal if this restriction remains, but it didn't seem useful to me at the time, even in practice: you can command ETB through TS without a pedal in various ways.

Feel free to close if you don't want to see it change.

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