Skip to content

Conversation

@FBruzzesi
Copy link
Member

@FBruzzesi FBruzzesi commented Jul 26, 2025

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

Opening as draft since we need to decide what to do for the cases in which the original dtype is non nullable for pandas (see #2890 (comment))

@dangotbanned
Copy link
Member

I hope its okay I linked the polars PR that added the null behavior.

I was looking to see it came from a feature request or had an origin in pandas for (#2890 (comment)) - but the search continues! 😅

@FBruzzesi
Copy link
Member Author

FBruzzesi commented Nov 10, 2025

As pointed in the commit message "keep it simple for now", for now I am failing the case of n>0 for both pandas-like and dask. All other backend should be ok to review for now .

Update: 3006fec is an attempt for pandas-like backends that allows to clear(n) only if it does not change the underlying dtype.

@FBruzzesi FBruzzesi marked this pull request as ready for review November 10, 2025 22:58
@FBruzzesi FBruzzesi changed the title feat: Add support for DataFrame.clear method feat: Add support for {DataFrame,LazyFrame}.clear method Nov 10, 2025
@FBruzzesi FBruzzesi changed the title feat: Add support for {DataFrame,LazyFrame}.clear method feat: Add support for {DataFrame,LazyFrame,Series}.clear method Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request needs discussion

Projects

None yet

4 participants