Skip to content

fix: 修复chroma_store.py中unsupported-operand-type问题 #2702

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

china-zhz
Copy link
Contributor

@china-zhz china-zhz commented May 17, 2025

Description

在使用Chroma vector store没有指定ChromaVectorConfig中的persist_path属性时,项目启动会报错如下:
1747457384407
其中chroma_vector_config.get("persist_path", os.path.join(PILOT_PATH, "data")) 执行结果是None而不是os.path.join(PILOT_PATH, "data")的值,因为字典类型的get方法只有当不包含此键时才会使用指定的默认值,但是chroma_vector_config字典中persist_path键值存在只不过值为None

How Has This Been Tested?

使用Chroma vector store ->新增一个数据库 -> 项目启动 问题可复现,测试流程也如此

Checklist:

  • [√] My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • [√] I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

@github-actions github-actions bot added the fix Bug fixes label May 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant