gpxpy icon indicating copy to clipboard operation
gpxpy copied to clipboard

Typo: type_of_gpx_fix should be type_of_gps_fix ?

Open gregoryng opened this issue 5 years ago • 1 comments

Minor typo shows up in a few places in gpx.py. There is a field called "type_of_gpx_fix", but I think it should be called "type_of_gps_fix." I assume it is referring to "type of gps fix", which the topografix manual has also made this error in one spot, but generally refers to this as "GPS fix".

I guess it is consistent in the code to internally call it "type_of_gpx_fix" rather than "type_of_gps_fix", but technically it doesn't quite make sense.

I see 8 occurrences in gpx.py but this is the first: https://github.com/tkrajina/gpxpy/blob/4b90ead7163008953f2fd0ebb998e86320bc9f84/gpxpy/gpx.py#L66

It also shows up in test.py.

Don't know if the right response is to allow either field or to deprecate type_of_gpx_fix.

gregoryng avatar Jun 26 '20 16:06 gregoryng

Yes, you're right. Thanks for pointing this out.

I'm also not sure how to fix it, but I'll think of something.

tkrajina avatar Jul 09 '20 20:07 tkrajina