CircularDependencyError should return the graph nodes/edges as Python objects along with the error message

Issue #1890 resolved
Former user created an issue

Hi,

Yesterday I had to debug a circular dependency error, and having to make sense of a cca. 150 KByte long error message was not an easy task. Had the CircularDependencyError object contained the real graph nodes/edges and not just their repr(), I could have used e.g. python-graph to visualize the data, which would have made spotting the error much more easier.

Gabor

Comments (4)

  1. Mike Bayer repo owner
    • changed component to orm
    • changed milestone to 0.6.4

    are you on 0.6 at least ? the topological structure has been dramatically simplified. e.g.:

    CircularDependencyError: Circular dependency detected: cycles: set('node3', 'node2', 'node5', 'node4', ('node4', 'node1'), ('node2', 'node3'), ('node1', 'node2'), ('node4', 'node5'), ('node5', 'node4')](('node3',)

    but sure we could stick those two members onto the exception.

  2. Log in to comment