Hi Bharath,

First of all, what should I do to add you as a collaborator to the
pull request (I was not aware it was possible)?

Second, we can meet on IRC or skype so I can help you with reviewing
the code. There is one big chunk of documentation missing and one
important change still needs to be done.

The documentation missing is about the Series factory class. It needs
to be explicitly mentioned that this is where most of the heuristics
are done when one wants to guess what type of plot the user wants.

The change that needs to be done is to add the more explicit plotting
routines. Those are easy one-liners. Aaron was worried that this means
that they will use the heuristic plot() function, but it is not the
case. They will just call the appropriate base series class and that
is all.

Before merging, the api should be stable. I think that the best way to
do this is to get the pull request in a polished stated using the
current api, then finish the wiki page with the examples and ask the
community for an opinion.

Can we meet on irc/skype today (I have exams this week so I won't be
able to dedicate more than an hour or so...)

Stefan

-- 
You received this message because you are subscribed to the Google Groups 
"sympy" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sympy?hl=en.

Reply via email to