indentation

[euhruska]

when running on a small problem I get:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "build/bdist.macosx-10.9-x86_64/egg/Christofides/christofides.py", line 164, in compute
  File "build/bdist.macosx-10.9-x86_64/egg/Christofides/christofides.py", line 120, in Euler_Tour
UnboundLocalError: local variable 'edge' referenced before assignment

I looks like the lines 120-123 in Christofides/christofides.py are wrongly indented with the for loop, maybe they should be indented with if command above.

[dsrahul30]

Thanks for the information! Looking into it.

…

On 21-Oct-2017 07:13, "Eugen Hruska" ***@***.***> wrote: when running on a small problem I get: Traceback (most recent call last): File "<stdin>", line 1, in <module> File "build/bdist.macosx-10.9-x86_64/egg/Christofides/christofides.py", line 164, in compute File "build/bdist.macosx-10.9-x86_64/egg/Christofides/christofides.py", line 120, in Euler_Tour UnboundLocalError: local variable 'edge' referenced before assignment I looks like the lines 120-123 in Christofides/christofides.py are wrongly indented with the for loop, maybe they should be indented with if command above. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#3>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AIbWHiHm8hjYC25YgfHqpfOXdDYr_IUQks5suUwrgaJpZM4QBZCm> .

[gagangoku]

Its a bug. Its not working even for the test cases mentioned in graph.py

[alfcar9]

SimonDeRidde has a pull request that works on that repository and fixes the bug.