|
From: Michael D. <md...@st...> - 2009-10-28 13:59:46
|
Eero Nevalainen wrote:
>> 2) forgot a factor 2 for the width and height (it's the entire width
>> not the `radius`)
>>
>
> I'd even say that this is a documentation bug in the Ellipse class.
> Too bad that they are multiplying by 0.5 inside their code :P
>
Well, it's not a good idea to change the existing behavior now, but we
can improve the documentation. What would you suggest? Would you
prefer to see the word "diameter" in there explicitly somehow?
It currently says:
*width*
length of horizontal axis
*height*
length of vertical axis
Mike
--
Michael Droettboom
Science Software Branch
Operations and Engineering Division
Space Telescope Science Institute
Operated by AURA for NASA
|