Changes between Initial Version and Version 3 of Ticket #8781


Ignore:
Timestamp:
05/10/10 17:35:02 (9 years ago)
Author:
ncohen
Comment:

Oops.... That's totally unrelated, and is caused by the recent upgrade of NetworkX. The default graphs from NetworkX (of which ClawGraph?() is an example) now have {} instead of None as default edge labels... And as {} is not hashable, Sage does not like to find it as part of a dictionary key ;-)

There was already a patch waiting for review to fix it, which is #8892, but I had forgotten this file and only fixed generic_graph and graph... Well, I updated that patch, which is now a dependency of this very one, and fixes the bug you found.

Thank you very much Minh, and sorry again for that :-)

Nathann

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #8781

    • Property Status changed from new to needs_review
    • Property Reviewers changed from to Minh Van Nguyen
    • Property Authors changed from to Nathann Cohen
  • Ticket #8781 – Description

    initial v3  
    33I also fixed a mistake in this code : I had mixed g.order() with max(g.degree()) for complete graphs ^^;
    44
     5Requires #8892
     6
    57Nathann