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

BUG: Arugment nan_as_null is unused in __dataframe__ calls. #61025

Open
3 tasks done
ethandavidlee opened this issue Mar 1, 2025 · 2 comments
Open
3 tasks done

BUG: Arugment nan_as_null is unused in __dataframe__ calls. #61025

ethandavidlee opened this issue Mar 1, 2025 · 2 comments
Assignees
Labels
Clean Closing Candidate May be closeable, needs more eyeballs Interchange Dataframe Interchange Protocol

Comments

@ethandavidlee
Copy link

Pandas version checks

  • I have checked that this issue has not already been reported.

  • I have confirmed this bug exists on the latest version of pandas.

  • I have confirmed this bug exists on the main branch of pandas.

Reproducible Example

Code review of frame.py (line 904).

Issue Description

The arugment nan_as_null is unused in dataframe calls (Line 904 in frame.py) and should be removed. A comment notes that "nan_as_null is DEPRECATED and has no effect. Please avoid using it; it will be removed in a future release". nan_as_null has now been removed but the argument is still present in the function call.

Image

The issue was last touched on in 54846. but this was before nan_as_null was supposed to be removed.

Expected Behavior

The argument can be removed if not in use for the sake of clean code.

Installed Versions

v3.0.0.dev0-1976-g6e61cf44ee

@ethandavidlee ethandavidlee added Bug Needs Triage Issue that has not been reviewed by a pandas team member labels Mar 1, 2025
@ethandavidlee
Copy link
Author

take

@rhshadrach
Copy link
Member

rhshadrach commented Mar 2, 2025

Thanks for the report. As mentioned in the linked PR, it is not to be removed for 3.0.

#54846 (comment)

Also, I think it should not be removed until the upstream issue is closed.

data-apis/dataframe-api#125 (comment)

Finally, pandas is planning to deprecate the entire protocol, so I think we can just leave this as is.

#56732 (comment)

I think we're good to add the deprecation for the entire protocol in 3.0, but checking in the above issue. If you'd like to work on that, it'd be a great help!

@rhshadrach rhshadrach added Closing Candidate May be closeable, needs more eyeballs Interchange Dataframe Interchange Protocol Clean and removed Bug Needs Triage Issue that has not been reviewed by a pandas team member labels Mar 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Clean Closing Candidate May be closeable, needs more eyeballs Interchange Dataframe Interchange Protocol
Projects
None yet
Development

No branches or pull requests

2 participants