Modify

Opened 3 years ago

Last modified 8 days ago

#21334 reopened defect

[PATCH] image direction is wrong when position is beetween 2 gpx point

Reported by: StephaneP Owned by: Don-vip
Priority: normal Milestone: 21.10
Component: Core image mapping Version:
Keywords: support-layer Cc: Don-vip

Description (last modified by StephaneP)

1) correlate images positions with a support layer, and one image timestamp is beetween two support-layer node
2) enable "override position" and "set image direction"

The image direction should be calculated with the image position and the next node, but it's calculated with the next segment direction. See this screenshot:


Attachments (2)

img_direction_and_support_layer.jpg (215.2 KB ) - added by StephaneP 3 years ago.
0001-Point-to-the-next-Waypoint.patch (1.7 KB ) - added by StephaneP 10 days ago.
fix direction

Download all attachments as: .zip

Change History (10)

by StephaneP, 3 years ago

comment:1 by StephaneP, 3 years ago

Description: modified (diff)

comment:2 by Don-vip, 3 years ago

Milestone: 21.09
Owner: changed from team to Don-vip
Status: newassigned

comment:3 by Don-vip, 3 years ago

Resolution: fixed
Status: assignedclosed

In 18237/josm:

see #21144 - fix #21334 - fix correlated image direction when position is between 2 gpx points

comment:4 by Don-vip, 3 years ago

In 18243/josm:

see #21144 - see #21334 - add debug logs + fix coverity 1464077 (Logically dead code: nextWp cannot be null with the isLast branch, but Eclipse doesn't see it)

comment:5 by Don-vip, 3 years ago

Milestone: 21.0921.10

Milestone renamed

comment:6 by StephaneP, 3 years ago

Resolution: fixed
Status: closedreopened

As #21364 this bug is back in r18321

comment:7 by StephaneP, 10 days ago

In the current code, when correlating, the image direction isn't pointing to the next Waypoint (curWp) but the 2nd next one (nextWp). I change this with:

-            LatLon nextCoorForDirection = nextWp.getCoor();
+            LatLon nextCoorForDirection = curWp.getCoor();

I then add a comparison in case image and curWp coordinates are equal, to change the nextCoorDirection to nextWp.

+if (nextCoorForDirection.equals(curCoor)) {
+                            nextCoorForDirection = nextWp.getCoor();
+                        }
Last edited 8 days ago by StephaneP (previous) (diff)

by StephaneP, 10 days ago

fix direction

comment:8 by StephaneP, 10 days ago

Summary: image direction is wrong when position is beetween 2 gpx point[PATCH] image direction is wrong when position is beetween 2 gpx point

Modify Ticket

Change Properties
Set your email in Preferences
Action
as reopened The owner will remain Don-vip.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from Don-vip to the specified user. Next status will be 'new'.
Next status will be 'needinfo'. The owner will be changed from Don-vip to StephaneP.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.