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
-~----------~----~----~----~------~----~------~--~---

Reply via email to