TrickfireRobotics / urc-2023

The codebase for the "Viator" rover for the University Rover Challenge
10 stars 2 forks source link

Add a Current Limit to All Moteus Controllers #52

Closed VladimirKupryukhin closed 4 hours ago

VladimirKupryukhin commented 1 month ago

Context

The Moteus Controllers have a config, and one of these options is to limit the current. We had most likely fried a controller in the past because there was no limit

What to do

Go through all controllers and put a reasonable current on the controllers. No more than 10A for sure; make sure you consider the operating current for the motor and the Moteus controller when selecting the limit

loukylor commented 1 week ago

The moteus python examples include an example on how to set config values. We'd have to expose this functionality somewhere in the moteus controllers. I was thinking something like add **kwargs for config values in the moteus controller constructor. However, we could also expose a set_config method since, according to the reference page, config values take effect immediately.

VladimirKupryukhin commented 1 week ago

As of right now, there are current limits to the following motors:

loukylor commented 4 hours ago

fixed by #62