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

Query engine logger #1037

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

fclarke1
Copy link

@fclarke1 fclarke1 commented Aug 27, 2024

Description

There is no query logger when using graphrag.query. I've implemented a query-global-engine.log and a query-local-engine.log in a very similar way to the indexing-engine.log
I found the query logger very helpful when debugging my graphrag implementation

Fix for Issue #995

Proposed Changes

  • query-local-engine.log and query-global-engine.log are saved in root/output/{latest_timestamp}/reports/
  • graphrag.query.cli now accepts --verbose argument
  • query logger is now enabled using enable_logging_with_config() in graphrag.query.cli

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).

@fclarke1 fclarke1 requested review from a team as code owners August 27, 2024 16:31
@fclarke1
Copy link
Author

@microsoft-github-policy-service agree

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