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

Add multi-graph tests for weighted similarity algorithms #4132

Open
Tracked by #3280
ChuckHastings opened this issue Jan 31, 2024 · 0 comments
Open
Tracked by #3280

Add multi-graph tests for weighted similarity algorithms #4132

ChuckHastings opened this issue Jan 31, 2024 · 0 comments
Assignees

Comments

@ChuckHastings
Copy link
Collaborator

Weighted similarity tests are currently disabled for RMAT. This is due to the fact that RMAT generates a multigraph, and the reference implementation for weighted similarity and the cugraph primitive-based implementation for weighted similarity generate different results for a multi-graph.

This needs to be investigated. I suspect the issue is that the desired behavior is not clear in the presence of a multi-graph. So we probably first need to decide what the right behavior is in the case of a multi-graph. Then we need to update the reference implementation to reflect the proper behavior. We might also need to implement the primitive-based implementation - unless the proper behavior is what's currently implemented by the primitive.

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

No branches or pull requests

1 participant