Skip to content

Added in newer Examples / Quickstarts #9

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

Closed
wants to merge 0 commits into from

Conversation

davidfauth
Copy link

Updated the Patient-Journey and TPC-H examples to work with the current Neo4j Graph Analytics for Snowflake.
Added in a WCC example based on Business / Location / Executives.

"name": "cell15"
},
"outputs": [],
"source": "CALL se_snow_neo4j_graph_analytics.graph.wcc('CPU_X64_L', {\n 'project': {\n 'defaultTablePrefix': 'WCC_EXAMPLE.PUBLIC',\n 'nodeTables': ['PERSON_VW','ORGANIZATION_VW','LOCATION_NODES_VW'], \n 'relationshipTables': {\n 'COMMITTEE_VW': {\n 'sourceTable': 'PERSON_VW',\n 'targetTable': 'ORGANIZATION_VW'\n },\n 'LOCATION_ORGANIZATION_VW': {\n 'sourceTable': 'LOCATION_NODES_VW',\n 'targetTable': 'ORGANIZATION_VW'\n }\n }\n },\n 'compute': {},\n 'write': [\n {\n 'nodeLabel': 'PERSON_VW',\n 'outputTable': 'WCC_EXAMPLE.PUBLIC.USERS_COMPONENTS'\n }\n ]\n});",
Copy link
Collaborator

Choose a reason for hiding this comment

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

it's better to use neo4j_graph_analytics name for the app

Comment on lines 254 to 255
"CREATE OR REPLACE STAGE sf_neo4j_data_stage\n",
" URL = 's3://neo4j-snowflake-data/tpc-h/';"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it the same data as here?
https://docs.snowflake.com/en/user-guide/sample-data-tpch

If yes, I think its better to use it from sf. In this data can be easily obtain with these queries without having a stage

-- Create a database from the share.
CREATE DATABASE SNOWFLAKE_SAMPLE_DATA FROM SHARE SFC_SAMPLES.SAMPLE_DATA;

-- Grant the PUBLIC role access to the database.
-- Optionally change the role name to restrict access to a subset of users.
GRANT IMPORTED PRIVILEGES ON DATABASE SNOWFLAKE_SAMPLE_DATA TO ROLE PUBLIC;

see https://docs.snowflake.com/en/user-guide/sample-data-using

},
"outputs": [],
"source": [
"CALL neo4j_graph_analytics.graph.node_similarity('CPU_X64_L', {\n",
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this notebook mirrors this file https://github.com/neo4j-product-examples/snowflake-graph-analytics/blob/main/product-recommendation/product-recommendation.sql

please have a look, you can find something helpful there

"source": [
"The Weakly Connected Components (WCC) algorithm finds sets of connected nodes in directed and undirected graphs. Two nodes are connected, if there exists a path between them. The set of all nodes that are connected with each other form a component.\n",
"\n",
"This example uses data that was sourced from `Relationship 360 - People Business Intelligence Sample (Massachusetts)`. This data set is a free dataset that is available in the Snowflake Marketplace."
Copy link
Collaborator

Choose a reason for hiding this comment

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

"name": "cell26"
},
"source": [
"When you write the tables back to the database, the tables are owned by the application. You will need to alter the tables so that subsequent post-processing or visualization is easier by avoiding joins to the node ID mapping tables necessary for GDS. To do this, you have to transfer ownership of the tables back to accountadmin role."
Copy link
Collaborator

Choose a reason for hiding this comment

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

we avoid using "GDS" acronym in favor to "Graph Analytics" term

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