Skip to content

CLI WP #4473

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

Merged
merged 3 commits into from
Mar 15, 2019
Merged

CLI WP #4473

merged 3 commits into from
Mar 15, 2019

Conversation

hali9
Copy link
Contributor

@hali9 hali9 commented Mar 8, 2019

At the moment there is no way to save the mission in inav or inav-configurator in a simple way like other settings. Sometimes it is also necessary to create a mission from another program or on the basis of a black box log.
This PR makes this possible.

@DzikuVx
Copy link
Member

DzikuVx commented Mar 8, 2019

Nice @hali9

But I can already say that you have to disable this for all F3 targets if you want to fit it

@stronnag
Copy link
Collaborator

stronnag commented Mar 8, 2019

What is the wp load use case? Once you hit save or exit (exit the CLI in order to arm) the WPs in volatile RAM are lost.

@hali9
Copy link
Contributor Author

hali9 commented Mar 8, 2019

@DzikuVx it fit, but I will disable it on F3.
@stronnag wp load loads WPs from non volatile memory, then You can type wp to list it or use diff or dump to copy it and save to file just like other settings.
To restore just paste in WPs (can be with other settings) in CLI and type wp save to save WPs to non volatile memory.
Normal save not saves WPs to non volatile memory must use wp save.
But wp save save WPs and also all settings so no need to use save save when using wp save.

Copy link
Collaborator

@shellixyz shellixyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shellixyz shellixyz added this to the 2.2 milestone Mar 13, 2019
@shellixyz shellixyz merged commit 5b25cfc into iNavFlight:development Mar 15, 2019
@hali9 hali9 deleted the dev-wp branch March 16, 2019 22:10
@OlivierC-FR
Copy link
Contributor

Help ! :)

wp reset
wp 1 1 459965100 59989293 400 0 0
wp 2 1 459916380 59984627 500 0 165
wp save

Parse error

@stronnag
Copy link
Collaborator

In the CLI, wp are 0 based.

@stronnag
Copy link
Collaborator

and will 4 or 5m really clear the trees?

@OlivierC-FR
Copy link
Contributor

Ok thanks Im going to try !
That's not to actually set a proper mission, it's to simulate nearby aircrafts, from the LoRa modem, for the iNav radar.

@OlivierC-FR
Copy link
Contributor

OlivierC-FR commented Mar 18, 2019

Ok it worked, it actually needed : wp 0 1 ...

@stronnag
Copy link
Collaborator

stronnag commented Mar 18, 2019

"0 based ..." :)
image

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.

5 participants