Skip to content

DEPR: to_clipboard, read_clipboard #56039

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

Closed
wants to merge 1 commit into from

Conversation

jbrockmendel
Copy link
Member

@WillAyd
Copy link
Member

WillAyd commented Nov 18, 2023

Why do we want to deprecate these? I personally find them useful every now and again - are these a maintenance burden?

@jbrockmendel
Copy link
Member Author

My main complaint is that we effectively aren't (and AFAICT can't) test it on the CI. If I'm wrong about it being rarely used then i wont mind keeping it around.

@WillAyd
Copy link
Member

WillAyd commented Nov 18, 2023

I would prefer to keep these. Sure not the main reason for using pandas, but they are nice for quick transfers of data across applications

@MarkRotchell
Copy link

I use this functionality all the time. Often will define something like the below at the top of a notebook for sending arbitraty array-likes to the clipboard as I'm modelling. Clearly, not something needed (much?) in production code, but when doing ad-hoc throw-away work/analysis, it's incredibly useful.

def clippy(data):
    pd.DataFrame(data).to_clipboard()

@jbrockmendel
Copy link
Member Author

I don't care about this deprecation enough to argue about it, but my thought is that the def clippy(data) use case would be better served by

def clippy(data):
    content = pd.DataFrame(data).to_csv()
    something_that_lives_outside_of_pandas.to_clipboard(content)

@mroeschke
Copy link
Member

My attempt to test clipboard in the CI in #56084

@phofl
Copy link
Member

phofl commented Nov 22, 2023

Merged the test pr, so closing here

@phofl phofl closed this Nov 22, 2023
@jbrockmendel jbrockmendel deleted the depr-clipboard branch November 22, 2023 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DEPR: to_clipboard
5 participants