-
Notifications
You must be signed in to change notification settings - Fork 5
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
Nav subcommand #33
Draft
johannessen
wants to merge
11
commits into
cr:master
Choose a base branch
from
johannessen:nav-command
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Nav subcommand #33
Commits on Oct 27, 2019
-
Fix unpack_waypoint() latitude parsing
The fifth byte in a waypoint definition is 0xf0 for waypoints created on the device itself, but 0xff for waypoints created in Yaesu's PC software, YCE15. The radio seems to ignore the fifth byte altogether, so I'm not sure whether that difference was intentionally introduced by Yeasu, or if it was a mistake. Either way, both formats should be accepted by unpack_waypoint(). This change accomplishes that by simply ignoring the fifth byte, starting unpacking with the sixth byte, which contains the most significant digits of the latitude value. In order to be in line with Yeasu's own use, pack_waypoint() is changed to use 0xff for the fifth byte. This doesn't seem to matter one way or the other though.
Configuration menu - View commit details
-
Copy full SHA for 8a81db3 - Browse repository at this point
Copy the full SHA 8a81db3View commit details -
Add nav command to dump waypoints read from device to GPX
A first step towards dumping/flashing navigation data. The code in nav.py is basically lifted straight from gpslog.py.
Configuration menu - View commit details
-
Copy full SHA for 8e48a85 - Browse repository at this point
Copy the full SHA 8e48a85View commit details -
Configuration menu - View commit details
-
Copy full SHA for c6a20ca - Browse repository at this point
Copy the full SHA c6a20caView commit details -
Configuration menu - View commit details
-
Copy full SHA for 191a7db - Browse repository at this point
Copy the full SHA 191a7dbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4453407 - Browse repository at this point
Copy the full SHA 4453407View commit details -
Configuration menu - View commit details
-
Copy full SHA for e8548d8 - Browse repository at this point
Copy the full SHA e8548d8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 358bf79 - Browse repository at this point
Copy the full SHA 358bf79View commit details -
The current and previously used navigation targets are implemented as indices to the waypoint/route lists. After replacing these lists with new data, those indices are no longer usable and may put the radio in an inconsistent state. This change works around this issue by clearing the navigation status and history for data read from GPX.
Configuration menu - View commit details
-
Copy full SHA for cbbe312 - Browse repository at this point
Copy the full SHA cbbe312View commit details -
Configuration menu - View commit details
-
Copy full SHA for be148d5 - Browse repository at this point
Copy the full SHA be148d5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3e24229 - Browse repository at this point
Copy the full SHA 3e24229View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2e86642 - Browse repository at this point
Copy the full SHA 2e86642View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.