-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[ENH] Implement literal expression for regex #4421
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] Implement literal expression for regex #4421
Conversation
Reviewer ChecklistPlease leverage this checklist to ensure your code review is thorough before approving Testing, Bugs, Errors, Logs, Documentation
System Compatibility
Quality
|
This stack of pull requests is managed by Graphite. Learn more about stacking. |
a9a95c8
to
af29420
Compare
8872466
to
3675914
Compare
9593947
to
df27c54
Compare
Implementation of Literal Expression for Regular Expressions This PR adds custom internal representations for regular expressions in the Rust codebase to be used with full text search for document filtering. The implementation includes a Key Changes: Affected Areas: This summary was automatically generated by @propel-code-bot |
Merge activity
|
## Description of changes _Summarize the changes made by this PR._ - Improvements & Bug fixes - N/A - New functionality - Implement custom internal representations for regular expression. They will be used with full text search index to filter documents. ## Test plan _How are these changes tested?_ - [ ] Tests pass locally with `pytest` for python, `yarn test` for js, `cargo test` for rust ## Documentation Changes _Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the [docs section](https://github.com/chroma-core/chroma/tree/main/docs/docs.trychroma.com)?_
Description of changes
Summarize the changes made by this PR.
Test plan
How are these changes tested?
pytest
for python,yarn test
for js,cargo test
for rustDocumentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs section?