Skip to content

Commit

Permalink
Replace number attribute with a range of values when generating merge…
Browse files Browse the repository at this point in the history
…d GPX
  • Loading branch information
spilth committed Sep 27, 2023
1 parent 94126f5 commit 991bb0c
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,8 @@ def post_merge_gpx_tracks():
]
)

# The GPX driver requires the "number" column to be an integer, not a string
merged_tracks["number"] = pandas.to_numeric(merged_tracks["number"])
if output_format == "gpx":
merged_tracks["number"] = range(1, len(merged_tracks) + 1)

with tempfile.TemporaryDirectory() as temp_dir:
merged_file = str(Path(temp_dir) / f"merged.{extension}")
Expand Down
5 changes: 5 additions & 0 deletions templates/merge-gpx-tracks.html
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,11 @@
</label>
</div>
{% endfor %}

<div class="alert alert-warning" role="alert">
Note: For GPX output the <code>number</code> attribute will be replaced with an incremental counter to
avoid missing or duplicate values.
</div>
</div>

<button type="submit" class="btn btn-primary">Merge Tracks</button>
Expand Down

0 comments on commit 991bb0c

Please sign in to comment.