-
Notifications
You must be signed in to change notification settings - Fork 155
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
Add filter for string contains case insensitive #816
Conversation
@marthacryan is attempting to deploy a commit to the trymito Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, it's working perfect! Hopefully you got a chance to poke around the backend a little bit as well.
A coupla tests we could add to cover our bases, and then feel free to merge into dev once tests pass (no need for another review)!
FC_STRING_CONTAINS_CASE_INSENSITIVE, | ||
"aBc", | ||
pd.DataFrame(data={"A": ["abcDEF", "ABCdef"]}), | ||
), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two more tests that might be nice:
- Combining two contains case insensitive with an And
- Combinion two contains case insensitive with an Or
Just so we cover all our cases here and make sure they are working properly!
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
4f65dbd
to
6028475
Compare
eee7df0
to
98ba82f
Compare
Adds a string filter condition for contains w/ case insensitive.
For comparison, these are the results with contains:
And these are with contains case insensitive: