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

Motor Setup #12306

Closed
wants to merge 2 commits into from
Closed

Motor Setup #12306

wants to merge 2 commits into from

Conversation

Godeffroy
Copy link

@Godeffroy Godeffroy commented Jan 8, 2025

Motor Setup

Change the motor setup page :
Instead of a slider for each motor with the difficulty to know how many power you put in, i made a slider to choose the power with fine tuning, a button for each motor to test them, a button to start all motor and a button to stop them all at once.


Tested on an hexa copter with Ardupilot Copter 4.5

Checklist:

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Change the slider to test each motors with button and slider combinaison
@Godeffroy
Copy link
Author

Godeffroy commented Jan 8, 2025

I commited all translation files because of the text changes. I don't know if i need to remove the *.ts files ? or do something to synchronize the translation ?

I also change the *.md file for the documentation with a new picture.

@HTRamsey HTRamsey requested a review from DonLakeFlyer January 8, 2025 19:20
@Godeffroy Godeffroy deleted the branch mavlink:master January 9, 2025 11:16
@Godeffroy Godeffroy closed this Jan 9, 2025
@Godeffroy Godeffroy deleted the master branch January 9, 2025 11:16
@Godeffroy
Copy link
Author

I somehow deleted my master branch because i wanted to work on another PR. I will push again this PR with another branch name instead of master. My bad :(

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

Successfully merging this pull request may close these issues.

1 participant