-
Notifications
You must be signed in to change notification settings - Fork 313
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
Initial PropertyGraph implementation and tests #1999
Initial PropertyGraph implementation and tests #1999
Conversation
…. Will need to add a pandas test later.
…2-initialpropgraph
…tests (type checking, property specification, etc.)
…ed FIXMEs and placeholder tests.
…2-initialpropgraph
…mports, added tests, added code for default_edge_weight, flake8 fixes.
…2-initialpropgraph
…rt of queries, added more query tests.
…rames, updated tests to parameterize dDataFrame type for both cuDF and Pandas, removed unnecessary PropertyColumn class, added more error checking and tests for correct input types, updated comments.
Codecov Report
@@ Coverage Diff @@
## branch-22.02 #1999 +/- ##
===============================================
Coverage ? 72.50%
===============================================
Files ? 146
Lines ? 9470
Branches ? 0
===============================================
Hits ? 6866
Misses ? 2604
Partials ? 0 Continue to review full report at Codecov.
|
rerun tests |
…ated edges before creating Graphs, allow for queries that don't have verts, more progress on annotate_dataframe(), added additional data to datasets to test multiple vert props and duplicated edges, refactored extract_subgraph() to allow for outside dataframes to generate graphs.
…is now passing, still need to update annotate_dataframe() to remove extra columns from other edge types, check args, add other tests.
…dates to handle Pandas differences in annotate_dataframe() with dtypes after merging.
…2-initialpropgraph
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.
Added a simple test for a completely empty graph but not sure it is worth adding during burn down. Can commit and push if wanted.
…2-initialpropgraph
…2-initialpropgraph
@gpucibot merge |
Initial implementation of the
PropertyGraph
graph class and corresponding unit tests.closes #1876