`ENVELOPE` method on linestring does not return `POLYGON` about rgeo HOT 5 OPEN

Got it you're using the Ruby implementation. I think since Geos is now returning either NULL a point or a polygon rather than a linestring as well like in older versions we can make this change to match. I can work on a fix in the next few days.

from rgeo.

What version of geos are you using? On 3.10 I'm seeing a polygon when I run this.

from rgeo.

I have not installed geos for this.
Correct me if i am wrong, but this method no where specifies explicit dependency on geos.
Also in the factory-compatibiltiy doc, LineString#envelope shows compatibilty with cartesian factory.

from rgeo.

@keithdoggett is there any update to this.
also can i help in fixing this.

from rgeo.

I've looked into it but haven't had time to write a PR. We need to change the envelope method in RGeo::Cartesian::GeometryMethods (

Per Geos it can either return a polygon or a point (https://github.com/libgeos/geos/blob/a8d2ed0aba46f88f9b8987526e68eea6565d16ae/capi/geos_c.h.in#L3881-L3887), so we should just add a check for the returned geometry from the BoundingBox call and if it's a linestring, coalesce it into a polygon (probably by taking the points into an array).

i.e. something like this (haven't run this)

def envelope
        env = BoundingBox.new(factory).add(self).to_geometry
        if RGeo::Feature::LineString.check_type(env)
           boundary = factory.line_string([env.start_point, env.end_point, env.start_point, env.end_point])
           env = factory.polygon(boundary)
       end
       env
end

from rgeo.