-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[ENH] Configuration UX #2437
[ENH] Configuration UX #2437
Conversation
Please tag your PR title with one of: [ENH | BUG | DOC | TST | BLD | PERF | TYP | CLN | CHORE]. See https://docs.trychroma.com/contributing#contributing-code-and-ideas |
Reviewer ChecklistPlease leverage this checklist to ensure your code review is thorough before approving Testing, Bugs, Errors, Logs, Documentation
System Compatibility
Quality
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
6ff9475
to
f53a3e3
Compare
e796bfc
to
4adbfb9
Compare
f53a3e3
to
3e1d005
Compare
4adbfb9
to
2a36bc0
Compare
3e1d005
to
5f9b4a5
Compare
2a36bc0
to
e0233d2
Compare
5f9b4a5
to
f046588
Compare
e0233d2
to
209e1f6
Compare
f046588
to
3f1bc3f
Compare
Description of changes
This PR creates a nicer user-facing constructor for collection configurations, wrapping the existing internal representation. This is basically hacky constructor polymorphism, everything internal remains the same.
Test plan
CI passes.
Documentation Changes
No changes.