Skip to content

[bugfix]Fix query error with --streaming#1368

Merged
AlonsoGuevara merged 5 commits into
microsoft:mainfrom
KylinMountain:astream
Nov 6, 2024
Merged

[bugfix]Fix query error with --streaming#1368
AlonsoGuevara merged 5 commits into
microsoft:mainfrom
KylinMountain:astream

Conversation

@KylinMountain

@KylinMountain KylinMountain commented Nov 6, 2024

Copy link
Copy Markdown
Contributor

Description

it will report error when running graphrag query --method local --query "what is the activity" --streaming

Related Issues

[Reference any related issues or tasks that this pull request addresses.]

Proposed Changes

fix the param conf_args to config_args when calling _get_embedding_store

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

Additional Notes

[Add any additional notes or context that may be helpful for the reviewer(s).]

@KylinMountain
KylinMountain requested review from a team as code owners November 6, 2024 09:46
@AlonsoGuevara
AlonsoGuevara merged commit baa261c into microsoft:main Nov 6, 2024
@KylinMountain
KylinMountain deleted the astream branch March 27, 2025 03:29
Brandsma pushed a commit to ThalamusLabs/MMGraphRAG that referenced this pull request Nov 6, 2025
* fix streaming output error

* add semversioner

---------

Co-authored-by: Alonso Guevara <alonsog@microsoft.com>
JonasReuter pushed a commit to JonasReuter/graphrag that referenced this pull request Apr 13, 2026
* fix streaming output error

* add semversioner

---------

Co-authored-by: Alonso Guevara <alonsog@microsoft.com>
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