Hi Nicolas, Thank you for figuring out what the problem was!! However, did you push your fix? With all patches applied, I still get the error and on the patch server I cannot see that you pushed any changes recently.
Cheers, Anne >> 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 --~--~---------~--~----~------------~-------~--~----~ 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 -~----------~----~----~----~------~----~------~--~---
