Skip to content

Remove lancedb_dir redundant assignments#1163

Merged
AlonsoGuevara merged 2 commits intomicrosoft:mainfrom
longyunfeigu:fix/remove_lancedb
Sep 19, 2024
Merged

Remove lancedb_dir redundant assignments#1163
AlonsoGuevara merged 2 commits intomicrosoft:mainfrom
longyunfeigu:fix/remove_lancedb

Conversation

@longyunfeigu
Copy link
Contributor

@longyunfeigu longyunfeigu commented Sep 19, 2024

Description

lancedb_dir = lancedb_dir = Path(config.storage.base_dir) / "lancedb"
=> lancedb_dir = Path(config.storage.base_dir) / "lancedb"

Related Issues

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

Proposed Changes

[List the specific changes made in this pull request.]

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

@longyunfeigu longyunfeigu requested review from a team as code owners September 19, 2024 09:02
@AlonsoGuevara AlonsoGuevara merged commit 95409ff into microsoft:main Sep 19, 2024
opensourcemukul pushed a commit to opensourcemukul/graphrag that referenced this pull request Sep 13, 2025
Co-authored-by: wanhua.gu <wanhua.gu@wiz.ai>
Co-authored-by: Alonso Guevara <alonsog@microsoft.com>
Brandsma pushed a commit to ThalamusLabs/MMGraphRAG that referenced this pull request Nov 6, 2025
Co-authored-by: wanhua.gu <wanhua.gu@wiz.ai>
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