Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#2206 closed defect (wontfix)

[raster]: ST_HillShade why did we switch to degrees in 2.1

Reported by: robe Owned by: Bborie Park
Priority: medium Milestone: PostGIS 2.1.0
Component: raster Version: master
Keywords: Cc:


I propose we switch back before release. I'm flagging this for 2 reasons

1) It's a breaking change from 2.0 2) We dinged pramsey for trying to sneak in use of degrees in his ST_Project function.

Our general convention is to use radians unless if there is another over-aching standard that requires degrees and just demonstrate in the docs how to flip between the two using the built in PostgreSQL degrees and radians functions as demonstrated in:


Change History (3)

comment:1 by Bborie Park, 11 years ago

If you look at the docs for ST_Slope, ST_Azimuth and ST_Hillshade in 2.0, there was absolutely no units described. So, when I was going through the code to make sure the results matched that of ArcGIS (I assume most folks are going to compare to that application), I noticed that ArcGIS uses degrees by default. And since our functions where not documented, I went with degrees…

Personally, I like degrees…

comment:2 by robe, 11 years ago

Resolution: wontfix
Status: newclosed

Okay I'm fine with that. Hopefully pramsey isn't listening cause then he'll want his degrees back.

Note: See TracTickets for help on using tickets.