Giter Club home page Giter Club logo

Comments (3)

cpesch avatar cpesch commented on June 2, 2024

Hi watchingJu,

that's actually 2 effects:

  1. As you already found out, the GPX file is invalid as it uses the wrong version attribute value. Whoever produces such files should be notified.
  2. Upon failing to reading invalid GPX 1.0 and 1.1 RouteConverter tries to be more relaxed to read crappy files. This produces the funny output from the screenshot. I don't see an issue here since it's clearly visible that the input data is broken.

Best regards
Christian

from routeconverter.

watchingJu avatar watchingJu commented on June 2, 2024

Hello Christian,

thanks for your fast response.

  1. Yes, the first thing I did when finding that bug.
  2. When something in software is visibly broken it let me, as a tech person, interpret it in 2 ways:
    a) the input data is broken
    b) the output data (software) is broken

Some background:
I used this file in different programs and online tools before in search of a fitting tool, I did surprisingly not ran into this problem (means the other programs are less strict/accurate with parsing - a decision in design). This software (RouteConveter) did not produce any error, or other visible user feedback than a broken output, so my first thought was b) - either code wise (wrong parsing), or UX wise - no user feedback, that something went wrong while processing the data.

So my suggestion is - give some user feedback (alert box, message in the status bar,... like "Error in the input data, trying to parse the file anyway") that the program knows something is wrong with the input data, so the user knows it is not the program itself.

  1. I guess the "400 response" because of the " at the end is either wrong parsing, or wrong logs (?).

Cheers,
Ju

from routeconverter.

cpesch avatar cpesch commented on June 2, 2024

Thank you for pointing out a bug in the decision logic whether the greedy Garmin POI Database format has correctly identified a file.

You can find the latest prerelease with the fix here: https://www.routeconverter.com/prereleases/en

from routeconverter.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.