Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

C extension implementation returns paths with more hops #1

Open
glciampaglia opened this issue Sep 11, 2016 · 0 comments
Open

C extension implementation returns paths with more hops #1

glciampaglia opened this issue Sep 11, 2016 · 0 comments
Assignees
Labels

Comments

@glciampaglia
Copy link
Owner

glciampaglia commented Sep 11, 2016

When multiple paths with the same proximity are available, sometime the Cython implementation (cclosuress) returns paths with more hops than the pure Python implementation (closuress).

This bug was surfaced in knowledge_linker.tests.test_closure.test_closure_rand; it may be a Cython specific issue since regression shows it was introduced in 31589e6 (464d864fd1979229a8f5412f31ae04a5282d5848 on Github.IU), but it seems that the test started failing only recently when a newer version of Cython was used to regenerate the C extension modules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant