Modify

Opened 6 years ago

Closed 5 years ago

#5730 closed Bugs (fixed)

strategy selection error in the distance function

Reported by: Christian Marty <christian.marty@…> Owned by: barendgehrels
Milestone: Boost 1.48.0 Component: geometry
Version: Boost 1.47.0 Severity: Problem
Keywords: geometry distance strategy Cc:

Description

file : \boost\geometry\algorithms\distance.hpp

line : 134

before (bug ?) : return_type const ds = ps_strategy.apply(point, *prev, *it);

after (patched) : return_type const ds = eps_strategy.apply(point, *prev, *it);

from the line 125, 128 and 139 eps_strategy seems the more appropriate.

Attachments (0)

Change History (3)

comment:1 Changed 6 years ago by barendgehrels

  • Milestone changed from To Be Determined to Boost 1.48.0

There is indeed something wrong here. But it is not as simple as changing this line only. Will look at it soon.

comment:2 Changed 6 years ago by mloskot

Is this new report related #5954 ?

comment:3 Changed 5 years ago by barendgehrels

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

This was indeed a problem. Actually it was more severe than I first thought. A patch was supplied recently and is applied now. The additional issue was that after that patch the unit test did not run anymore, and that is fixed as well. Sorry that this was delayed.

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain barendgehrels.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.