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

Replaced looptime with gyro/pid denom settings #26

Merged
merged 1 commit into from Jun 20, 2016
Merged

Replaced looptime with gyro/pid denom settings #26

merged 1 commit into from Jun 20, 2016

Conversation

KiteAnton
Copy link
Contributor

As requested in #22
Looptime is replaced by gyro/pid denom settings

@borisbstyle borisbstyle merged commit a950bdd into betaflight:master Jun 20, 2016
@borisbstyle
Copy link
Member

Excellent!

Do you think you can add rc_yaw_rate and maybe the filters into PID tab?

@KiteAnton KiteAnton deleted the replace_looptime branch June 20, 2016 20:10
@KiteAnton
Copy link
Contributor Author

@borisbstyle, do you mean MSP_SET_FILTER_CONFIG and MSP_SET_ADVANCED_TUNING ? I can have a look at it but maybe someone else are better in adjusting the layout later.

@borisbstyle
Copy link
Member

Arent those easiest to add? Just a field like pids etc. No special logic
behind
Op 20 jun. 2016 22:25 schreef "KiteAnton" notifications@github.com:

@borisbstyle https://github.com/borisbstyle, do you mean
MSP_SET_FILTER_CONFIG and MSP_SET_ADVANCED_TUNING ? I can have a look at it
but maybe someone else are better in adjusting the layout later.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#26 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AKQlhDgwY4sJelA5qm2XZJAKnu7HQmUJks5qNvcdgaJpZM4I6Dt1
.

@skaman82
Copy link
Contributor

No worries, I can allways clean up the GUI later.

Von meinem iPhone gesendet

Am 20.06.2016 um 22:25 schrieb KiteAnton notifications@github.com:

@borisbstyle, do you mean MSP_SET_FILTER_CONFIG and MSP_SET_ADVANCED_TUNING ? I can have a look at it but maybe someone else are better in adjusting the layout later.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

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.

None yet

3 participants