search

ArduPilot / MissionPlanner

Mission Planner Ground Control Station for ArduPilot (c# .net)
http://ardupilot.org/planner/
GNU General Public License v3.0
1.69k stars 2.33k forks source link

Request: Survey (Grid) - Allow polygon boundary to be exceeded by 1 flight line. #572

Closed iskess closed 9 years ago

iskess commented 9 years ago

I would like the Survey tool to allow the first or last flight line to exceed the boundary wall to prevent the awkward triangle that is formed when there is only enough lateral space to fit part of the flight line. This issue could also be helped by allowing the polygon to be adjusted while inside the Survey tool (#571).

tristannovak commented 9 years ago

@iskess I am not sure I agree... (I may just not understand the concept, could you post an example?).

I am not sure how many manual editing abilities we want to have in the survey tool, when they are already available and meant to be in the flight plan tool...

p.s. I just added left mouse PAN to my pull request.

tristannovak commented 9 years ago

I can't remember what issue number it was but I also just fixed the border and footprint colors. And added it to pull request #569

colors

iskess commented 9 years ago

Ok here is a simple example: survey grid

You can see that WP10 hits the border which results in shortened flight line. To correct this, we could either allow manual editing of the polygon border to provide more space for the flight line to continue, or allow the last flight line to pass through the border. I think the most intuitive would be to allow manual editing of the polygon inside the tool so you could watch the flight line redraw in real time, just like we do when we adjust 'Angle [deg]".

tristannovak commented 9 years ago

I can see how that would be useful, I may have even tried to do that when I was first learning MP... I will look into what it would take to make it possible.

note to devs - It will also need to update the original poly in the flightplanner map..

Looking at that image I think I will also look into making the "return" path dashed or different than the regular paths in someway. (just needed to write down the though so I don't forget.)

tristannovak commented 9 years ago

Added the ability to manually Edit the polygon, should take care of this issue. #576.