Support for the postgis adapter #117
Conversation
| POW(#{lng_degree_units}*(#{origin.lng}-#{qualified_lng_column_name}),2)) | ||
| | | ||
| end | ||
|
|
| def flat_distance_sql(origin, lat_degree_units, lng_degree_units) | ||
| %| | ||
| SQRT(POW(#{lat_degree_units}*(#{origin.lat}-#{qualified_lat_column_name}),2)+ | ||
| POW(#{lng_degree_units}*(#{origin.lng}-#{qualified_lng_column_name}),2)) |
|
|
||
| def flat_distance_sql(origin, lat_degree_units, lng_degree_units) | ||
| %| | ||
| SQRT(POW(#{lat_degree_units}*(#{origin.lat}-#{qualified_lat_column_name}),2)+ |
| SIN(#{lat})*SIN(RADIANS(#{qualified_lat_column_name}))))*#{multiplier}) | ||
| | | ||
| end | ||
|
|
| %| | ||
| (ACOS(least(1,COS(#{lat})*COS(#{lng})*COS(RADIANS(#{qualified_lat_column_name}))*COS(RADIANS(#{qualified_lng_column_name}))+ | ||
| COS(#{lat})*SIN(#{lng})*COS(RADIANS(#{qualified_lat_column_name}))*SIN(RADIANS(#{qualified_lng_column_name}))+ | ||
| SIN(#{lat})*SIN(RADIANS(#{qualified_lat_column_name}))))*#{multiplier}) |
| def sphere_distance_sql(lat, lng, multiplier) | ||
| %| | ||
| (ACOS(least(1,COS(#{lat})*COS(#{lng})*COS(RADIANS(#{qualified_lat_column_name}))*COS(RADIANS(#{qualified_lng_column_name}))+ | ||
| COS(#{lat})*SIN(#{lng})*COS(RADIANS(#{qualified_lat_column_name}))*SIN(RADIANS(#{qualified_lng_column_name}))+ |
|
|
||
| def sphere_distance_sql(lat, lng, multiplier) | ||
| %| | ||
| (ACOS(least(1,COS(#{lat})*COS(#{lng})*COS(RADIANS(#{qualified_lat_column_name}))*COS(RADIANS(#{qualified_lng_column_name}))+ |
| module Geokit | ||
| module Adapters | ||
| class PostGIS < Abstract | ||
|
|
| @@ -0,0 +1,22 @@ | |||
| module Geokit | |||
There was a problem hiding this comment.
Missing frozen string literal comment.
1 similar comment
|
Why use PostGIS if you don't use the provided functions ST_Distance ST_Distance_Sphere... |
|
Sorry @babasbot for not getting around to this. I've rebuilt the PR so it's now passing, the last remaining issues stopping me merging it are that it reduces coverage (so we should have a test to ensure this code doesn't just work now, but works well into the future). And finally it would be good to address the comment made by another user here. But otherwise I'm happy to merge. |
|
Sorry @babasbot - Closing this as it hasn't been active. Feel free to write the test case and I'm happy to re-look at it (and most likely merge it) |
|
I too ended up forking this repo just to add the duplicate |
The activerecord-postgis-adapter provides access to features of the PostGIS geospatial database from ActiveRecord. It extends the standard postgresql adapter to provide support for the spatial data types and features added by the PostGIS extension.
Changelog
postgisadapter.Closes #116