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

[BUG] Flatten on projected fragment yields incorrected result in GAE #2476

Closed
sighingnow opened this issue Mar 1, 2023 · 1 comment · Fixed by #2477
Closed

[BUG] Flatten on projected fragment yields incorrected result in GAE #2476

sighingnow opened this issue Mar 1, 2023 · 1 comment · Fixed by #2477
Assignees
Labels
bug Something isn't working component:gae priority:high Issues in high priority

Comments

@sighingnow
Copy link
Collaborator

Describe the bug

import graphscope
from graphscope.dataset import load_ldbc

graph = load_ldbc()

pg = graph.project(vertices={"post": [], "tag":[]}, edges={"hasTag": []})

pr_context = graphscope.pagerank_nx(pg, alpha=0.85, max_iter=100, tol=1e-06)
print(pr_context.to_dataframe(
    selector={"id": "v.id", "dist": "r"}
).sort_values(by="id"))

pg = graph.project(vertices={"post": [], "tag":[]}, edges={"hasTag": []})
c = graphscope.hits(pg)
print(c.schema)
c.to_dataframe(
    selector={"id": "v.id", "auth": "r.auth", "hub": "r.hub"}
).sort_values(by="id")

The result contains vertex not in "post" and "tag", and the result set size is wrong.

Additional context

Reported by @hanjf12

@sighingnow sighingnow added bug Something isn't working component:gae priority:high Issues in high priority labels Mar 1, 2023
@sighingnow sighingnow self-assigned this Mar 1, 2023
@sighingnow
Copy link
Collaborator Author

See also: v6d-io/v6d#1223

sighingnow added a commit that referenced this issue Mar 7, 2023
## What do these changes do?

- Fixes bugs in flatten fragment
- Upgrade GAE to latest vineyard

## Related issue number

Fixes #2476

See also:
- #211
- v6d-io/v6d#239
- v6d-io/v6d#1224

---------

Signed-off-by: Tao He <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working component:gae priority:high Issues in high priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant