Ticket #63 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

ST_Max_Distance is not yet implemented?

Reported by: robe Owned by: robe
Priority: medium Milestone: PostGIS 1.5.0
Component: postgis Version: 1.4.X
Keywords: Cc: nicklas

Description (last modified by pramsey) (diff)

What steps will reproduce the problem? SELECT ST_Max_Distance(ST_GeomFromText('LINESTRING(1 0, 3 0)'), ST_GeomFromText('LINESTRING(3 0, 5 0)'))

What is the expected output? huh I think 4

What do you see instead?

ERROR: This function is unimplemented yet

********** Error **********

ERROR: This function is unimplemented yet SQL state: XX000

What version of the product are you using? On what operating system? Windows - 'POSTGIS='1.3.3' GEOS='3.0.0-CAPI-1.4.1' PROJ='Rel. 4.6.0, 21 Dec 2007' USE_STATS' I think I tried on OpenSUSE 11 1.3.4 SVN and got the same unsatisfying answer. I'll have to double-check that.

Please provide any additional information below. I think the trunk lwpostgis.sql.in.c has this comment

Maximum distance between linestrings. 2d only. Very bogus. -- Availability: 1.2.2

And I thought it did something way back yander, but could have been my imagination or it was removed because as the comment says its bogus?

Attachments

max_distance.patch Download (56.3 KB) - added by nicklas 3 years ago.
maxdist_doc.patch Download (0.9 KB) - added by nicklas 3 years ago.
Document-patch for max_distance function
shortest_longest_fullywithin_new_poly_poly_20090717.patch Download (51.8 KB) - added by nicklas 3 years ago.
max_distance_new_poly_poly_20090717.patch Download (1 bytes) - added by nicklas 3 years ago.
max_distance_new_poly_poly.patch Download (39.6 KB) - added by nicklas 3 years ago.

Change History

Changed 3 years ago by robe

Someone needs to look at Nicklas patch he attached in #137 for max distance to see how far off he is from a working solution. Would be nice to see this working in 1.4.1 and possibily 1.3.7.

Changed 3 years ago by post...@…

I have made a new try on st_maxdistance and now I think it is working as I want. The last try was very bad. There is a new patch in #137

/Nicklas Avén

Changed 3 years ago by nicklas

Changed 3 years ago by nicklas

Document-patch for max_distance function

Changed 3 years ago by nicklas

in max_distance_new_poly_poly_20090717.patch

there is a big rewrite of polygon-polygon handling in distance-measurement more about it in #137

Changed 3 years ago by nicklas

Changed 3 years ago by nicklas

Changed 3 years ago by nicklas

Changed 2 years ago by pramsey

  • description modified (diff)

If the implementation of this is bound up with the other distance fixes, please move this to milestone 1.5 and this issue will just have to remain in 1.4 as is. I'd also like to rename to ST_MaxDistance() in 1.5.

Changed 2 years ago by pramsey

  • cc nicklas added

Changed 2 years ago by pramsey

  • milestone changed from postgis 1.4.1 to postgis 1.5.0

I'm moving this to 1.5 in anticipation that it will merge with #231

Changed 2 years ago by pramsey

  • status changed from assigned to closed
  • resolution changed from accepted to fixed

Brought in with distance work at r4894 from Nicklas Aven's work. Note that function name has been changed to ST_MaxDistance()

Note: See TracTickets for help on using tickets.