-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Fix: Address multiple issues #4998
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
base: main
Are you sure you want to change the base?
Fix: Address multiple issues #4998
Conversation
Reviewer ChecklistPlease leverage this checklist to ensure your code review is thorough before approving Testing, Bugs, Errors, Logs, Documentation
System Compatibility
Quality
|
Multi-Issue Fix: OpenTelemetry Version Sync, Thin Client Config, Metadata Safety, Embedding Consistency, and Minor Logic Fixes This PR introduces a series of targeted fixes and improvements across the codebase. It synchronizes all OpenTelemetry dependencies to version 1.24.0 to prevent version conflict, makes Key Changes: Affected Areas: This summary was automatically generated by @propel-code-bot |
opentelemetry
package versions to1.24.0
inpyproject.toml
andrequirements.txt
to resolve version conflicts.is_thin_client
a configurable setting within theSettings
class inchromadb/config.py
for explicit control._records
function inchromadb/api/segment.py
to prevent unintended side effects.normalize_embeddings
function inchromadb/api/types.py
to avoid type mismatches.uris
in theadd
method ofCollection.py
andAsyncCollection.py
.test_runtime_dependencies
inchromadb/test/test_config.py
to ensure components are registered before system startup.Test Plan
pytest
(manual verification of code changes and logical correctness).Documentation Changes