Hi, sorry for the late review. I am on airport now going to Los Alamos, so I may have sporadic net connection in the coming days.
+ SYMBOL_SWAPPED = False I would use symbol_swapped. Could you also please add a comment into the code next to symbols_swapped about its purpose? E.g. that it is used to swap symbols for Derivative/Function, so that one can solve for it. Otherwise it looks ok to me. Ondrej On Mon, May 25, 2009 at 9:22 AM, Luke <hazelnu...@gmail.com> wrote: > > Has anybody had a chance to review this? If there is anything that > should be done differently, I'm happy to change it up, just let me > know. > Thanks, > ~Luke > > On May 21, 6:58 pm, Luke <hazelnu...@gmail.com> wrote: >> Attached is the code that allows you to use solve with equations that >> have Symbol, Function, or Derivative instances in them. >> >> It passed all tests on my machine. Let me know if there are any problems. >> >> I will be working on a similar patch to do the same thing for diff() >> >> Thanks, >> ~Luke >> >> 0001-Implemented-code-to-allow-for-solving-for-Function-a.patch >> 6KViewDownload > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "sympy-patches" group. To post to this group, send email to sympy-patches@googlegroups.com To unsubscribe from this group, send email to sympy-patches+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sympy-patches?hl=en -~----------~----~----~----~------~----~------~--~---