[Bf-python] Good old euler problem

[Martin Poirier]

No constants please.

If people want to multiply by a constant, they can
define it themselves.

If we have to provide anything, it should be a
function.

Martin

--- Campbell Barton <cbarton at metavr.com> wrote:

> Hey joseph,
> Whilst I see your intentions are good, I think this
> could be more of a 
> pain then a blessing...
> 
> better just to have constants
> Blender.Mathutils.RAD2DEG and DEG2RAD  - 
> then the users knows that when dealing with blender
> its ALWAYS RAD and 
> they can convert the value to get if they want.
> 
> 
> Joseph Gilbert wrote:
> > I agree that all radians would be fine.
> > 
> > The methods for each class make the assumption
> that the object is using 
> > degrees. So if you fill it with radians you will
> end up with a problem. 
> > The same can be true when we fill a euler with
> degrees after this fix 
> > and then call a method of the euler.
> > 
> > I have thought about this alot and I think that it
> would be nice to 
> > implement 2 types of eulers. One that uses radians
> and one that uses 
> > degrees. The conversion between the 2 would be a
> method of the class.
> > Euler.toRadians()
> > Euler.toDegrees()
> > In the class there would be a flag set to tell
> which type it's using for 
> > it's internal methods. This might also mean
> separating the constructors 
> > as REuler() and DEuler() or something.
> > 
> > I don't know where else this occurs, however,
> Euler seems particularly 
> > problematic.
> > 
> > On Tue, 20 Feb 2007 13:53:31 -0500, Ken Hughes
> <khughes at pacific.edu> wrote:
> > 
> >> In the post-2.43 release glow, I wanted to float
> the issue of our 
> >> radian/degree issue in the Mathutils module and
> elsewhere:
> >>
> >>
>
https://projects.blender.org/tracker/index.php?func=detail&aid=3362&group_id=9&atid=264
> 
> >>
> >>
> >> I really think we should fix this for the next
> release, and I think 
> >> the best thing to do is implement a fix early in
> the cycle so that we 
> >> can be sure scripts are converted as necessary.
> >>
> >> I think the prevailing decision was to go with
> radians in the API, not 
> >> just in the Mathutils module.  So should this
> also be applied to the 
> >> Ipo module (should rotation Ipocurves return real
> radians instead of 
> >> the degree/10 stuff)?  That's bitten me a few
> times lately.
> >>
> >> Ken
> >> _______________________________________________
> >> Bf-python mailing list
> >> Bf-python at projects.blender.org
> >>
>
http://projects.blender.org/mailman/listinfo/bf-python
> > 
> > 
> > 
> 
> 
> -- 
> Campbell J Barton
> 
> 133 Hope Street
> Geelong West, Victoria 3218 Australia
> 
> URL:    http://www.metavr.com
> e-mail: cbarton at metavr.com
> phone: AU (03) 5229 0241
> _______________________________________________
> Bf-python mailing list
> Bf-python at projects.blender.org
>
http://projects.blender.org/mailman/listinfo/bf-python
> 



 
____________________________________________________________________________________
Do you Yahoo!?
Everyone is raving about the all-new Yahoo! Mail beta.
http://new.mail.yahoo.com