graphrag icon indicating copy to clipboard operation
graphrag copied to clipboard

[bug fix]Fix strategy config in entity_extraction

Open KylinMountain opened this issue 1 year ago • 1 comments

Description

When I config strategy with nltk type for entity_extraction, it doesn't take effect.

Related Issues

Proposed Changes

  • read strategy from config and set it to EntityExtractionConfig, default is None which is aligned the definition of the field strategy.
  • auto file format using poetry run poe format to fix error report

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

KylinMountain avatar Jul 12 '24 07:07 KylinMountain

@AlonsoGuevara do you know what may cause this unit test fail? Actually, it's succeed in my local Mac, it is weird.

azurite --blobHost 127.0.0.1 --blobPort 10000
poetry run poe test_unit

image

KylinMountain avatar Jul 13 '24 01:07 KylinMountain

Merged in #1046

AlonsoGuevara avatar Aug 28 '24 23:08 AlonsoGuevara