Add rogue run finder

[dwj300]

Also add virtualennv / .DS_Store to the .gitignore

[tomstitt]

+1

[tomstitt]

+1

[tomstitt]

I think it would be better to split segments with large jumps into multiple segments, that way they don't need to be skipped. In that case a track with 1 segment that includes a skip:

<trk>
  <name>Lunch Ride</name>
  <type>1</type>
  <trkseg>
   <trkpt lat="37.8350560" lon="-122.1949540">
    <ele>328.0</ele>
    <time>2017-05-21T19:16:51Z</time>
   </trkpt>
   <trkpt lat="37.8350850" lon="-122.1948800">
    <ele>327.4</ele>
    <time>2017-05-21T19:16:52Z</time>
   </trkpt>
   <trkpt lat="37.8350570" lon="-122.1948800">
    <ele>327.5</ele>
    <time>2017-05-21T19:16:55Z</time>
   </trkpt>
   <!--Let's say the skip occurs here-->
   <trkpt lat="37.8350320" lon="-122.1949230">
    <ele>327.7</ele>
    <time>2017-05-21T19:16:57Z</time>
   </trkpt>
   <trkpt lat="37.8350100" lon="-122.1949850">
    <ele>327.8</ele>
    <time>2017-05-21T19:16:59Z</time>
   </trkpt>
   <trkpt lat="37.8350050" lon="-122.1950210">
    <ele>327.9</ele>
    <time>2017-05-21T19:17:00Z</time>
   </trkpt>
  </trkseg>
</trk>

Becomes:

<trk>
  <name>Lunch Ride</name>
  <type>1</type>
  <trkseg>
   <trkpt lat="37.8350560" lon="-122.1949540">
    <ele>328.0</ele>
    <time>2017-05-21T19:16:51Z</time>
   </trkpt>
   <trkpt lat="37.8350850" lon="-122.1948800">
    <ele>327.4</ele>
    <time>2017-05-21T19:16:52Z</time>
   </trkpt>
   <trkpt lat="37.8350570" lon="-122.1948800">
    <ele>327.5</ele>
    <time>2017-05-21T19:16:55Z</time>
   </trkpt>
  </trkseg>
  <!--End last segment, start new one-->
  <trkseg>
   <trkpt lat="37.8350320" lon="-122.1949230">
    <ele>327.7</ele>
    <time>2017-05-21T19:16:57Z</time>
   </trkpt>
   <trkpt lat="37.8350100" lon="-122.1949850">
    <ele>327.8</ele>
    <time>2017-05-21T19:16:59Z</time>
   </trkpt>
   <trkpt lat="37.8350050" lon="-122.1950210">
    <ele>327.9</ele>
    <time>2017-05-21T19:17:00Z</time>
   </trkpt>
  </trkseg>
</trk>

edit: Yes, gpxpy supports updating the gpx objects then saving them back as xml:

with open(file, "r") as f:
    gpx = gpxpy.parse(f)
track = gpx.tracks[0]
segment = track.segments[0]
if has_jump(segment):
    track.segments = ...
with open(file, "w") as f:
    file.write(gpx.to_xml())

There's an assumption in load_gpx that there is only one segment but that's easy to change

[dwj300]

good idea! ill make those changes today

[tomstitt]

Can you rename rogue to something more descriptive (split, remove_jumps, etc) and add a line to the readme? Also, what was your motivation for moving the .py files out of the top level? I don't think of this as something ppl will pip (since it's just some scripts) so I'm not sure it's needed