Nice!

Here are patches for the quick and easy stuff.  They can be applied to
the branch you posted.  I also noticed some other small issues, and they
are corrected in the patches.

> 4) please use explicit imports in the example. Then it would be clear
> that the import:
> 
> from sympy.physics.secondquant import *
> 
> doesn't import AntiSymmetricTensor.
> 
I agree that explicit imports is better code, so lets change it. (The
attached patches fix that.)  

But anyway, shouldn't AntiSymmetricTensor be imported by 'import *'?  In
my own code branch the example runs without error, so why doesn't it run
in your branch?  

I'll write doctests, but that will take me some time.

Øyvind

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

Attachment: 0001-Explicit-import-in-coupled-cluster-example.patch
Description: application/mbox

Attachment: 0002-Whitespace-errors-tabs-spaces.patch
Description: application/mbox

Attachment: 0003-Replaces-generic-exception-with-new-Exception-object.patch
Description: application/mbox

Attachment: 0004-Removes-output-from-test-procedures.patch
Description: application/mbox

Reply via email to