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

support for waypoints? #1

Open
tve opened this issue Feb 21, 2018 · 9 comments
Open

support for waypoints? #1

tve opened this issue Feb 21, 2018 · 9 comments

Comments

@tve
Copy link

tve commented Feb 21, 2018

Have you tried entering waypoints from a GPX file into a hex dump? I'd like to enter a couple dozen waypoints into the radio and don't really feel like doing it manually...

@cr
Copy link
Owner

cr commented Mar 2, 2018

Hey @tve,

I started looking into waypoints and wrote the required low-level codecs and it seems to be working fine. If you can live with a strict CSV-based format, I can add an import/export function for you.

However, I am wondering whether this would work for you as I am not aware of you Python skill level. At this point the hx870 tool is still rather experimental and thus geared towards Python hackers who are comfortable with figuring out the handset's TTY device, changing the code accordingly, and then hacking along in an IPython session (essentially knowing what they're doing). Although certainly on the mid-term roadmap, I won't have any time soon to make the tool really end-user friendly with nifty shell argument parsing, automatic device detection, etc.

@cr
Copy link
Owner

cr commented Apr 11, 2019

@tve, there's another feature request for waypoint export in issue #6 now, and I'm trying to get the discussion started about which export format to choose for a start. Have you perhaps formed an opinion on that by now?

@johannessen
Copy link
Contributor

I’ve started work to address this issue over at johannessen/nav-command. Dumping waypoints and routes from the radio to GPX is already implemented. I hope to add flashing from GPX to the radio within the next few days.

The new functionality is accessible through a new CLI command for hxtool. I propose to use the term nav because it is short and a common abbreviation for the umbrella term “navigation”, which the HX870 manual uses for waypoints and routes.

Proposed CLI (not yet implemented):

hxtool nav -g data.gpx --dump
hxtool nav -g data.gpx --flash  # append GPX data to existing nav data in radio
hxtool nav -g data.gpx --flash --erase  # replace existing nav data in radio with GPX data
hxtool nav --erase  # erase all nav data from radio

For starters, I’d like to concentrate just on GPX, as it is easy enough to work with and can combine waypoints and routes in a single file. Other formats, such as plain text, CSV or GeoJSON, could conceivably be added later, in a similar fashion to the existing gpslog command.

@cr, do you happen to have any unpushed local changes this work might conflict with? Anything else you think I should rework, or consider going forward? Any feedback is appreciated.

@cr
Copy link
Owner

cr commented Oct 26, 2019

do you happen to have any unpushed local changes this work might conflict with?

Definitely, let me land these first before we move on.

@cr
Copy link
Owner

cr commented Oct 26, 2019

@johannessen, have a look at #29 that just landed on master. I hope it doesn't break too much of your code. I think the new classes should be easier to work with.

@cr
Copy link
Owner

cr commented Oct 26, 2019

I like the proposal to bundle all waypoint functionality under a nav subcommand. The GPS log-related subcommand could at some point be renamed to log, and I don't think clarity would suffer.

@cr
Copy link
Owner

cr commented Oct 26, 2019

I second the focus on GPX. It it a nasty format, but I think gpxpy supports GPX 1.0 well enough to easily work with waypoints as well.

@johannessen
Copy link
Contributor

The GPS log-related subcommand could at some point be renamed to log, and I don't think clarity would suffer.

The primary function of the radio is still making calls. A future subcommand named log should therefore probably be used for accessing the DSC call log rather than the GPS position log.

@johannessen
Copy link
Contributor

Thanks for pushing #29. No changes affecting nav subcommand. Will work on this next.

johannessen added a commit to johannessen/pyhx870 that referenced this issue Oct 27, 2019
johannessen added a commit to johannessen/pyhx870 that referenced this issue Oct 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants