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

Paramstatus: Display current status of received parameters to help with param fetch #674

Merged
merged 3 commits into from Aug 26, 2019

Conversation

@Akshath-Singhal
Copy link
Contributor

commented Aug 24, 2019

@tridge @peterbarker This displays parameter status both in console and in parameditor modules to prevent user from requesting parameter fetch repeatedly as discussed earlier.

PS : Three different commits for three different modules as suggested earlier.

@tridge

This comment has been minimized.

Copy link
Contributor

commented Aug 24, 2019

great! I'd like to see the commit msgs prefixed with module name, eg:
"console: display param status"

@Akshath-Singhal Akshath-Singhal force-pushed the Akshath-Singhal:paramstatus branch 2 times, most recently from d692091 to fe72818 Aug 24, 2019

@Akshath-Singhal

This comment has been minimized.

Copy link
Contributor Author

commented Aug 24, 2019

Done

Paramedit: Display param status
Display received parameters and total parameters

@Akshath-Singhal Akshath-Singhal force-pushed the Akshath-Singhal:paramstatus branch from fe72818 to 489f196 Aug 24, 2019

@tridge tridge merged commit d1c0ca2 into ArduPilot:master Aug 26, 2019

1 check failed

continuous-integration/appveyor/pr AppVeyor build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.