Divide by zero error if start and turn point 1 are identical about jsigc HOT 2 OPEN

I suppose the best thing we can do in this situation is either:

• Use the take-off position as the start and draw the start line there; or
• Use TP1 as the start and TP2 as TP1.

I'm not really sure which way would make more sense.

from jsigc.

On 13/10/15 18:13, Alistair Green wrote:

I suppose the best thing we can do in this situation is either:

• Use the take-off position as the start and draw the start line
  there; or
• Use TP1 as the start and TP2 as TP1.

I'm not really sure which way would make more sense.

—
Reply to this email directly or view it on GitHub
#15 (comment).

I don't think take off position will work. Declared take-off positions
often aren't in the file, and the actual take-off could be just about
any direction from the first declared point.

I must admit my instinct was just to refuse to draw the line but make
sure the track still displays- at the same time telling the user there's
an error. If we aren't happy with that the best I can suggest is to go
through the declared points probably at the end of parsing the file, and
if we find two successive points the same throw one away. This would
also help if two successive turning points are identical- as it is now
that doesn't throw an error, but the sector display is garbage.
Probably best done at the end of the igc parse function- because we've
already converted to floating point using checking for very small
differences.

Seem reasonable?

Regards

from jsigc.