Skip to content

Conversation

@amullick-git
Copy link
Collaborator

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@amullick-git amullick-git requested review from a team as code owners March 20, 2025 21:49
@product-auto-label product-auto-label bot added api: spanner Issues related to the googleapis/langchain-google-spanner-python API. samples Issues that are directly related to samples. labels Mar 20, 2025
@amullick-git amullick-git changed the title Chore(samples): Move inline data to be hosted on githbub repo for graphrag notebook chore(samples): Move inline data to be hosted on githbub repo for graphrag notebook Mar 20, 2025
"* Quick-drying surface prevents smudging\n",
"\n",
"Specifications:\n",
"!git clone https://github.com/amullick-git/retail_data\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 would recommend not linking out to other repos and storing the data directly in this repo, either in the docs/ or samples/ directory like retail_data/data.zip.
Then you can call in a code block before this one:

!wget https://github.com/googleapis/langchain-google-spanner-python/blob/main/samples/retail_data/data.zip
!unzip /content/data.zip

Then you can continue with the loading from the directory.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thank you for the suggestion. Made the change with slight modifications.

@averikitsch averikitsch merged commit a517e6e into googleapis:main Mar 21, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the googleapis/langchain-google-spanner-python API. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants