graphrag icon indicating copy to clipboard operation
graphrag copied to clipboard

[Bug#546]update key from community_reports to community_report

Open ronchengang opened this issue 1 year ago • 1 comments

Description

[Provide a brief description of the changes made in this pull request.]

In graphrag.config.create_graphrag_config.py, it refers to a wrong key name for the community report section of settings.yaml, it causes the problem that the community report config was never read into the program.

Related Issues

bug#546

Proposed Changes

[List the specific changes made in this pull request.] file path: graphrag.config.create_graphrag_config.py line #: 442 community_report_config = values.get("community_reports") or {} change to: ommunity_report_config = values.get("community_report") or {}

Checklist

  • [x] I have tested these changes locally.
  • [x] 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).]

ronchengang avatar Jul 14 '24 03:07 ronchengang

I also found this bug

gavin2014 avatar Jul 17 '24 01:07 gavin2014

I can see they have changed the default template of settings.yaml, this also works. This bug is fixed.

ronchengang avatar Jul 19 '24 13:07 ronchengang

No longer needed as this has been fixed

AlonsoGuevara avatar Jul 25 '24 04:07 AlonsoGuevara