On Thu, Aug 27, 2009 at 11:57:57AM -0700, Brant Jones wrote:
> I just downloaded the 4.1.1 binary and ran ./sage -combinat install,
> so I should be working with a fresh version of everything.
> Nevertheless, I still get that the 2 node is connected to the 5 node
> in E7 when I run:
>
> C = CartanType(['E',7])
> C.dynkin_diagram()
Thanks.
> However, when I do
>
> C.dynkin_diagram().show()
>
> then I see the 2 node connected to the 4 node! So, I guess the
> problem is probably in the terminal printing function?
Definitely.
And argl ... I take a tripe blame here:
- First, I wrote the broken code (or more precisely broke Dan's code
while refactoring it)
- Second, I forgot that multiplicity of spaces are ignored by sage -t.
Hence the tests I had put in type_E.py were not catching the bug.
- Third I actually had not the 4326 patch applied on my machine,
which is why I could not reproduce the problem.
I just pushed a fix. Thanks much for the report, and sorry for the
time it took.
Cheers,
Nicolas
--
Nicolas M. Thiéry "Isil" <nthi...@users.sf.net>
http://Nicolas.Thiery.name/
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-combinat-devel" group.
To post to this group, send email to sage-combinat-devel@googlegroups.com
To unsubscribe from this group, send email to
sage-combinat-devel+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/sage-combinat-devel?hl=en
-~----------~----~----~----~------~----~------~--~---
