Skip to content

fix: .clip should not treat 0 and None as same - #2889

Merged
FBruzzesi merged 1 commit into
mainfrom
fix/clip-with-zero
Jul 26, 2025
Merged

FBruzzesi merged 1 commit into
mainfrom
fix/clip-with-zero

Conversation

@FBruzzesi

Copy link
Copy Markdown
Member

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

  • 💾 Refactor
  • ✨ Feature
  • 🐛 Bug Fix
  • 🔧 Optimization
  • 📝 Documentation
  • ✅ Test
  • 🐳 Other

Related issues

Spotted while narwhalifying great-tables

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 fix high priority Your PR will be reviewed very quickly if you address this labels Jul 26, 2025
@FBruzzesi
FBruzzesi requested a review from MarcoGorelli July 26, 2025 09:46
@FBruzzesi FBruzzesi added pyarrow Issue is related to pyarrow backend pandas-like Issue is related to pandas-like backends labels Jul 26, 2025
@FBruzzesi
FBruzzesi merged commit ee06d33 into main Jul 26, 2025
@FBruzzesi
FBruzzesi deleted the fix/clip-with-zero branch July 26, 2025 09:53
@MarcoGorelli

Copy link
Copy Markdown
Member

nice, thanks!

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 pandas-like Issue is related to pandas-like backends pyarrow Issue is related to pyarrow backend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants