Skip to content
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

fix: Allow exprs in .filter constraints #2114

Merged
merged 2 commits into from
Mar 1, 2025

Conversation

FBruzzesi
Copy link
Member

What type of PR is this? (check all applicable)

  • πŸ’Ύ Refactor
  • ✨ Feature
  • πŸ› Bug Fix
  • πŸ”§ Optimization
  • πŸ“ Documentation
  • βœ… Test
  • 🐳 Other

Related issues

Checklist

  • Code follows style guide (ruff)
  • Tests added
  • Documented the changes

If you have comments or can explain your changes, please do so below

@FBruzzesi FBruzzesi added the fix label Feb 28, 2025
@FBruzzesi FBruzzesi added the high priority Your PR will be reviewed very quickly if you address this label Feb 28, 2025
@dangotbanned dangotbanned self-requested a review February 28, 2025 12:20
@dangotbanned
Copy link
Member

dangotbanned commented Feb 28, 2025

@FBruzzesi
The range wouldn't let me add as a suggestion 😞
Copy link
Member

@dangotbanned dangotbanned left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this @FBruzzesi

Feel free to revert (497dedf) before merging if you prefer the original

@FBruzzesi
Copy link
Member Author

Thanks for this @FBruzzesi

Feel free to revert (497dedf) before merging if you prefer the original

Thanks @dangotbanned - I like your approach πŸš€πŸ™πŸΌ

@FBruzzesi FBruzzesi changed the title RFC, fix: allow exprs in filter constraints fix: allow exprs in filter constraints Feb 28, 2025
@FBruzzesi FBruzzesi changed the title fix: allow exprs in filter constraints fix: Allow exprs in .filter constraints Mar 1, 2025
@FBruzzesi FBruzzesi merged commit dc9fcaa into main Mar 1, 2025
28 of 30 checks passed
@FBruzzesi FBruzzesi deleted the fix/filter-by-expr-constraints branch March 1, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix high priority Your PR will be reviewed very quickly if you address this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: df.filter(c=nw.col('a')) gives incorrect result for pandas and unclear error message for duckdb
2 participants