Opened 4 years ago

Closed 3 years ago

#418 closed defect (fixed)

Coordinates changing from West to East when saving

Reported by: vyseman Owned by: umair
Priority: major Component: Qt-UI
Version: 4.0.2 Keywords: Map, Coordinates, easyfix
Cc: Plattform (please give specifics in the ticket text): Windows

Description

Hi guys. I recently downloaded version 4.0.2 and am experiencing some wierd behaviour when saving a new new dive site. Basically I want to save a site in the north eastern quadrant (Bay of Pigs, Cuba to be exact) with the following coordinates: N22° 6.49440' , W81° 6.84540'.

I am able to find the place on the globe/map but when I save the dive, the coordinates immediately changes to N22° 6.49440' , E81° 6.84540'. I have played with other coordinates and this behaviour seems consistent, upon saving west changes to east (except for some place in Florida, whuch is kind of strange). I believe this is a major issue as it would preclude any dives from being registered in, say, the US or South America.

Change History (7)

comment:1 Changed 4 years ago by DirkHH

Which language do you run Subsurface in?

comment:2 Changed 4 years ago by DirkHH

A better question would be, in your locale, which letters are used for North South East and West?

comment:3 Changed 4 years ago by pleglise

Hi guys,

I have the exact same trouble. I leave in Martinique (in the Caribean sea). My last dive was here :
N14° 38.67942' , W61° 8.54496'
I'm french, so the language is set to "French". If I choose a location on the map, the software set the coordinates to N14° 38.67942' , O61° 8.54496'
In french, west is "Ouest" and east is "Est". So, the "O" is correct. But when I save, it sets the coordinates to N14° 38.67942' , E61° 8.54496', which is in the middle Arabian sea...
If I try to chage it manually to "O" or to "W", it keeps setting it to the east...
So I tryed to change the language to "English", then set it manually to the west with a "W" and it worked !! It saved it at the right position.
Went back to french and the coordinates are still ok...
Any clue ?

comment:4 Changed 3 years ago by DirkHH

  • Keywords easyfix added

This must be a parsing bug. Should be easy to reproduce and fix

comment:5 Changed 3 years ago by DirkHH

  • Owner changed from tcanabrava to no one
  • Status changed from new to assigned

comment:6 Changed 3 years ago by umair

  • Owner changed from no one to umair

dirrkhh I have take the responsiblity to fix this bug as a part of gsoc project proposal

comment:7 Changed 3 years ago by Dirk Hohndel <dirk@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

In 917ce5aff50370e7dc4805920a4f5eec09884bb3/subsurface:

Fix gps parsing

The argument index used to pull in the translated hemisphere letters was
incorrect.

Fixes #418

Signed-off-by: Gopichand Paturi <gopichandpaturi@…>
Signed-off-by: Dirk Hohndel <dirk@…>

Note: See TracTickets for help on using tickets.