-
Notifications
You must be signed in to change notification settings - Fork 82
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
H-2205, H-2354: Implement multi-type entities in the graph #4140
H-2205, H-2354: Implement multi-type entities in the graph #4140
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4140 +/- ##
==========================================
- Coverage 27.56% 27.52% -0.04%
==========================================
Files 278 278
Lines 11600 11613 +13
Branches 1174 1177 +3
==========================================
- Hits 3197 3196 -1
- Misses 8387 8401 +14
Partials 16 16
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥳
🌟 What is the purpose of this PR?
Multi-type entities enables entities to have multiple type without creating a new type.
🔍 What does this change?
EntityTypeMetadata
to store a vector of type idsPre-Merge Checklist 🚀
🚢 Has this modified a publishable library?
This PR:
📜 Does this require a change to the docs?
The changes in this PR:
🕸️ Does this require a change to the Turbo Graph?
The changes in this PR:
🐾 Next steps
🛡 What tests cover this?