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

Io to clipboard #3845

Merged
merged 4 commits into from
Jun 13, 2013
Merged

Io to clipboard #3845

merged 4 commits into from
Jun 13, 2013

Conversation

nipunbatra
Copy link
Contributor

Added documentation for to_clipboard().
Closes #3784

@hayd
Copy link
Contributor

hayd commented Jun 11, 2013

Is it worth adding a note in the APIs about system requirements aka #3837 (not if below pull is merged.)


.. ipython:: python

obj=pd.read_clipboard()
Copy link
Contributor

Choose a reason for hiding this comment

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

you don't need to asssign here...just pd.read_clipboard() (its like ipython, will just print the results)

also...these can all be in a single ipython:: python directive

and giong to need to change the note to conform with #3848

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do i need to send another PR? or can this be edited?

@hayd
Copy link
Contributor

hayd commented Jun 13, 2013

@nipunreddevil If you update your banch that will be reflected here.

@nipunbatra
Copy link
Contributor Author

OK. Whats the note to be now put in the documentation?

On Thu, Jun 13, 2013 at 5:18 PM, Andy Hayden [email protected]:

@nipunreddevil https://github.com/nipunreddevil If you update your
banch that will be reflected here.


Reply to this email directly or view it on GitHubhttps://github.com//pull/3845#issuecomment-19386728
.

@nipunbatra
Copy link
Contributor Author

Updated to put code in one ipython block and note reflects documentation from docstring.


.. note::

You may need to install xlip or xsel (with gtk or PyQt4 modules) on Linux to use these methods.
Copy link
Contributor

Choose a reason for hiding this comment

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

should be xclip

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure. Correcting that. Had also by mistake put some text at page top.
Removing that too.
@hayd Had copied from #3848, so there also "xlip" needs to be changed to
"xclip" in the docstring :)

On Thu, Jun 13, 2013 at 6:50 PM, jreback [email protected] wrote:

In doc/source/io.rst:

+the clipboard. Following which you can paste the clipboard contents into other
+applications (CTRL-V on many operating systems). Here we illustrate writing a
+DataFrame into clipboard and reading it back.
+
+.. ipython:: python
+

  • df=pd.DataFrame(randn(5,3))
  • df
  • df.to_clipboard()
  • pd.read_clipboard()

+We can see that we got the same content back, which we had earlier written to the clipboard.
+
+.. note::
+

  • You may need to install xlip or xsel (with gtk or PyQt4 modules) on Linux to use these methods.

should be xclip


Reply to this email directly or view it on GitHubhttps://github.com//pull/3845/files#r4678338
.

@hayd
Copy link
Contributor

hayd commented Jun 13, 2013

Thanks for spotting the typo!

@nipunbatra
Copy link
Contributor Author

Fixed the typo. The docs still take ever so long to build!

@jreback
Copy link
Contributor

jreback commented Jun 13, 2013

you should rebase on master, that prob was fixed a few commits ago

git co master
git pull
git br yourbranch --set-upstream master
git co yourbranch
git pull --rebase

hayd added a commit that referenced this pull request Jun 13, 2013
@hayd hayd merged commit e7488f1 into pandas-dev:master Jun 13, 2013
@nipunbatra nipunbatra deleted the io-to-clipboard branch June 14, 2013 02:01
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.

DOC: add to_clipboard example in io.rst
3 participants