⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@gaudyb
Copy link
Contributor

@gaudyb gaudyb commented Feb 4, 2026

Description

Add multiple example notebooks to each package for better user support.

Checklist

  • I have tested these changes locally.
  • I have reviewed the code changes.
  • I have updated the documentation (if necessary).
  • I have added appropriate unit tests (if applicable).

@gaudyb gaudyb requested review from dworthen and natoverse February 4, 2026 17:32
@gaudyb gaudyb requested a review from a team as a code owner February 4, 2026 17:32
@dworthen
Copy link
Contributor

dworthen commented Feb 6, 2026

Hi @gaudyb, Looks like the notebooks for graphrag_llm were moved to example_notebooks. Can you update tests/notebook/test_notebooks.py with the new location? It appears the test is currently passing when there are no notebooks to test. Can you update the test so that an error is thrown if notebooks_list is empty so that we don't have false positives when the tests don't run anything.

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.

3 participants