search

mongoid / mongoid-geospatial

A MongoDB/Mongoid Extension with optional GeoRuby/RGeo support.
http://rubygems.org/gems/mongoid-geospatial
MIT License
80 stars 31 forks source link

Require rgeo-geojson with rgeo for easier conversions and wider support? #67

Open camiloforero opened 6 years ago

camiloforero commented 6 years ago

As the title says, I propose that if an user wants to install this gem with rgeos support, they will also be required to install the rgeos-geojson package.

If we do that, then we can replace the whole rgeo.rb wrapper by calls to this library, which has more built-in robustness and a variety of functions already implemented.

And this will make it much easier to add support to MultiPoint, MultiLineString and MultiPolygon classes in the future, as they are already supported by that library.

Would this kind of change and additional optional dependency be ok?

dblock commented 6 years ago

Sounds good to me.

camiloforero commented 6 years ago

Cool! I can start working on that then