Re: BUG #6556: earthdistance module has lacking documentation

From: Kasper Sandberg <kontakt(at)sandberg-consult(dot)dk>
To: Kevin Grittner <Kevin(dot)Grittner(at)wicourts(dot)gov>
Cc: pgsql-bugs(at)postgresql(dot)org
Subject: Re: BUG #6556: earthdistance module has lacking documentation
Date: 2012-03-28 23:36:48
Message-ID: 4F73A090.5000101@sandberg-consult.dk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Yeah I will try provide a patch.

I am not entirely certain how the wording should be, but I only
understood that the cube based aproach is by default in meters after
testing it out, though now of course it occurs to me that of course it
is, given the math involved and that earth() is in meters, but i still
feel that it should be pointed out that the cube based functions will,
by default, due to earth() using meters, be in meters.

I will sleep on it a day or two and suggest a patch.

On 28/03/12 23:43, Kevin Grittner wrote:
> <kontakt(at)sandberg-consult(dot)dk> wrote:
>
>> looking at
>> http://www.postgresql.org/docs/9.1/static/earthdistance.html
>>
>> it looks very scarce and missing, and conflicting, some things
>> operate based on miles, others meters. it also doesnt specify the
>> units used for earth_distance() which seems to be meters.
>
> Any chance you could submit a patch to the source file?:
>
> doc/src/sgml/earthdistance.sgml
>
> ... or, barring that, suggest particular language at particular
> line numbers in the source based on the gitweb page?:
>
> http://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=doc/src/sgml/earthdistance.sgml;h=f9002b235b83d8e2facd9164a66db46792f6e54a;hb=HEAD
>
> -Kevin

--
Kasper Sandberg
Sandberg Enterprises
+45 51944242
http://www.sandbergenterprises.dk

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message gcottenc 2012-03-29 07:36:22 BUG #6565: GEQO documentation discrepancy after 9.0 change?
Previous Message Kevin Grittner 2012-03-28 22:31:31 Re: BUG #6522: PostgreSQL does not start