Comment #16 on issue 339 by fab...@fseoane.net: Series and expand as functions http://code.google.com/p/sympy/issues/detail?id=339
I have some objections: - e is not a nice parameter name in series() method for the expression, as it can be confused with the E constant. In my opinion variables consisting an a char should be avoided. (maybe e -> expr) - no need to sympify twice x and point, see the code for Basic.series(), as it already does simpify those argument's - I don't think putting var in the tests is a good idea, since it appends a symbol to the global namespace, that can conflict with other symbols declared later. - Last test lacks assert - Please consider writing some documentation to put in doc/src/modules/series.txt, even if it's just a link to the docstring. - it does not seem to work for values of point other than 0. See issue #1334. A workaround for this might be to do the series expansion for x-point at 0. -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---