Skip to content

fix community context builder#783

Merged
AlonsoGuevara merged 2 commits into
mainfrom
fix/community-context-builder
Jul 31, 2024
Merged

fix community context builder#783
AlonsoGuevara merged 2 commits into
mainfrom
fix/community-context-builder

Conversation

@ha2trinh

Copy link
Copy Markdown
Contributor

Description

When we cut a batch of community reports in the multi-batch context builder mode (used in global search), the last community report (which doesn't fit into the current batch) was not getting added to the new batch. This PR fixes this bug and refactors the community report context builder module.

Related Issues

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

Proposed Changes

  • Fixed logic to add the overflowing report of the current batch into the next batch.
  • Refactored to separate dataframe manipulation from batching logic

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

@ha2trinh
ha2trinh requested a review from a team as a code owner July 31, 2024 00:07
@AlonsoGuevara
AlonsoGuevara merged commit 7e1529a into main Jul 31, 2024
@AlonsoGuevara
AlonsoGuevara deleted the fix/community-context-builder branch July 31, 2024 02:14
@KylinMountain

KylinMountain commented Aug 1, 2024

Copy link
Copy Markdown
Contributor

@AlonsoGuevara @ha2trinh
have you tested this PR? there must be something wrong. After I merged this PR, it can't query now and report such exception.

Shall I need re-index my data?

image

dayesouza pushed a commit that referenced this pull request Aug 1, 2024
fix and refactor community context builder

Co-authored-by: Alonso Guevara <alonsog@microsoft.com>
congbo added a commit to congbo/graphrag that referenced this pull request Aug 2, 2024
Brandsma pushed a commit to ThalamusLabs/MMGraphRAG that referenced this pull request Nov 6, 2025
fix and refactor community context builder

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.

4 participants