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

Feature Request: Disable Field Weakening at compile #689

Open
Buzut opened this issue Dec 12, 2023 · 2 comments
Open

Feature Request: Disable Field Weakening at compile #689

Buzut opened this issue Dec 12, 2023 · 2 comments

Comments

@Buzut
Copy link

Buzut commented Dec 12, 2023

Field weakening (and other features) can be very dangerous if badly tuned.

The VESC doesn't know what the hardware limits are and as such, an end-user could set it to high values resulting in the ESC bursting in flames if something goes wrong.

For this very reason, field weakening (and maybe MTPA) should have a flag in the config file that can remove the feature so that no operator error can occur if it's not supposed to be used.

In addition, it would also be nice to be able to cap the values that can be configured.

@nitrousnrg
Copy link
Contributor

For that goal what I do is setting an RPM limit, like this:
https://github.com/luna-cycle/bldc/blob/0fd93961d4d7fe479beb8278b7f322b03b339b8f/hwconf/other/hw_axiom.h#L321

It doesn't cover every FW failure mode but it does help limiting the amount of voltage that the motor can generate, which can kill the powerstage if pwm stops firing.

@Buzut
Copy link
Author

Buzut commented Dec 13, 2023

Hey @nitrousnrg, thank you for that.
I think it would still be a really good idea to just have a flag to opt out from some features, but while waiting for it, it sure helps making things safer 👍

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