Status: New
Owner: ----
Labels: Type-Defect Priority-Medium
New issue 2807 by moorepa...@gmail.com: Change .unit and .mag to proper
methods and better names.
http://code.google.com/p/sympy/issues/detail?id=2807
sympy.physics.mechanics
.unit and .mag properties of the Vector class are a bit confusing. Some
people might think it refers to the units (i.e., m/s, or rad/s, etc.).
Also, by decorating them with @property, it gives the appearance these
quantities are precomputed: v.mag gives the impression that mag is data
member of v, whereas v.mag() gives the impression that you are calculating
something about v. With them as properties, I could see somebody wanting
to do something like:
v.unit = ....
which wouldn't work, but from a new user point of view might seem the
intuitive way to go.
I think it might be easier to learn sympy.mechanics if these were
regular instance methods, renamed to .magnitude(self) and
.normalize(self).
--
You received this message because you are subscribed to the Google Groups
"sympy-issues" group.
To post to this group, send email to sympy-issues@googlegroups.com.
To unsubscribe from this group, send email to
sympy-issues+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/sympy-issues?hl=en.
