-
Notifications
You must be signed in to change notification settings - Fork 66
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
core: frontend: Implement motor test #1499
Conversation
7cefe49
to
9c42905
Compare
|
3179362
to
cc74bb4
Compare
we still need to able to revert the motor direction, along with a couple of other things. I intend to do this as it's own component in another pr |
cc74bb4
to
bdcc794
Compare
}, | ||
methods: { | ||
styleForMotorBar(value: number): string { | ||
const percent = (value - 1500) / 10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't be SERVOn_MIN
and SERVOn_MAX
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
motors don't use SERVOn_MIN/MAX
We can expose MOT_PWM_MIN/MAX in another slider, I think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there should probably at minimum be a TODO here on using TRIM params here, instead of hardcoded 1500
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't really have TRIM params for motors, though 😅
bdcc794
to
ef733ba
Compare
@Williangalvani can you rebase over master ? |
command: { | ||
type: MavCmd.MAV_CMD_COMPONENT_ARM_DISARM, | ||
}, | ||
target_system: 1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That will be affect by #1581, do you prefer to merge it before ?
ef733ba
to
3ced98e
Compare
3ced98e
to
d110f40
Compare
needs some cleaning up, and probably refactoring the the arm button into a component
tested with a servo in the pwm rail and 5v jumped to it