#351 closed defect (fixed)
Doc bug: Wrong return type for ST_Envelope()
Reported by: | pracine | Owned by: | robe |
---|---|---|---|
Priority: | medium | Milestone: | PostGIS 1.4.2 |
Component: | postgis | Version: | 1.4.X |
Keywords: | Cc: |
Description
Return type for St_Envelope() in the doc should be "geometry". It is now "boolean".
http://postgis.refractions.net/documentation/manual-1.4SVN/ST_Envelope.html
It is also not really clear in this page whether ST_Envelope() will return the bbox float4s version or the float8s version.
Change History (4)
comment:1 by , 15 years ago
Milestone: | PostGIS 1.4.1 → PostGIS 1.4.2 |
---|
comment:2 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 by , 15 years ago
I've applied the correct fix to trunk for this (i.e. use full double precision calculation rather than using the BBOX cache to perform geometry calculations). I also fixed up ST_Expand() which was also doing a similar thing.
Regina - please can you verify and update the ST_Envelope() documentation accordingly? The whole section about float4 should simple be removed, along with the reference to float4 in the first paragraph.
Does anyone feel that this should be patched back to 1.4 branch?
comment:4 by , 15 years ago
I think its best not to patch back as that is kind of a borderline violation of behaving differently in a micro release. I don't think output behavior should change in a micro (if its not absolutely wrong and it was correctly documented)
Fixed at r5070 - r5073 (also added an example which hopefully clarifies the float4/float8 question) -- fixed for both trunk and branch/1.4