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

Node property storage and querying #58

Merged
merged 8 commits into from
Oct 18, 2023

Conversation

alvaradoo
Copy link
Member

This commit involves new functionality and a few fixes:

  • The new functionality involves being able to convert an Arkouda DataFrame to a property graph by calling the PropGraph.add_node_properties() method. This creates a two-dimensional array at the back end that stores arrays of variable size for each column. The columns of the two-dimensional array in the back-end corresponds to the data types that the property graph can store.
  • To be able to store maps and two-dimensional arrays in the symbol table we generated two new symbol entries: SymEntry2D and MapSymEntry.
  • Instead of requiring the user to name their label and relationship dataframe columns "nodeIds" and "src" and "dst" for adding in labels and relationships, we just assume the first column are node ids for adding in labels, and the first two columns for adding in edges.
  • Some minor changes to documentation.

@alvaradoo alvaradoo requested a review from zhihuidu October 18, 2023 21:54
Copy link
Contributor

@zhihuidu zhihuidu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Square count is a new algorithm

@zhihuidu zhihuidu merged commit 7c773cd into main Oct 18, 2023
@alvaradoo alvaradoo deleted the node_property_storage_and_querying branch October 19, 2023 00:43
mdindoost pushed a commit that referenced this pull request Jul 20, 2024
mdindoost pushed a commit that referenced this pull request Dec 3, 2024
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

Successfully merging this pull request may close these issues.

2 participants