remove waypoint function

[BirgitvdS]

remove waypoint function

[thaije]

@BirgitvdS thanks for letting it now that this is a feature you would like to see. I don't doubt that other users would also like an easy manner to remove waypoints, but I'm not sure on the details of how it should work such that that it works for everyone as intended.

The method I would think of would be something like:

self.navigator.remove_waypoint(location=(3,4)) # removes waypoint (3,4) from the waypoints list and restarts the navigator

However, if the waypoint you want to remove is present multiple times in the waypoints list, should they all be removed, or only the first or last?

[BirgitvdS]

My suggestion would be something like this:
self.navigator.remove_waypoint(location=(3,4), n= all)
in which "location" is the location of the waypoint and "n" is which waypoints you want to remove
Options could be:
n = all --> remove all waypoints with that location from the list
n = first --> remove first waypoints with that location from the list (if location is not specified, just remove first waypoint in the list)
n = last --> remove last waypoints with that location from the list (if location is not specified, just remove last waypoint in the list)
n = int --> in which int is an interger that specified which waypoint should be removed. For example if n= 5 than either "remove 5th waypoint which location X" or "remove 5th waypoint of the list" (in case no location in specified)
This way the user can use the remove function and specify what to remove.
Would this be possible?

[thaije]

Hey Birgit, although the solution you described would be useful, it is also quiet complicated with a lot of options, with minimal improvement over the two methods you described yourself. Instead of making a generic function with a lot of options which users most likely will not able to find anyway, it is better to make it easy to change the waypoints and reset the navigator. This is already the case, and the two alternatives you described seem sufficiently easy to use (possibly wrapped in your own remove_waypoint(navigator, waypoint) function) and easy to customize to suit a users' needs, such that an additional function is not necessary in my opinion.