DOC: table keyword missing in the docstring for Series.plot() and DataFr... #7698
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
HTML doc for this can be found here: http://pandas.pydata.org/pandas-docs/stable/visualization.html#plotting-tables |
| table : boolean, Series or DataFrame, default False | ||
| If True, draw a table using the data in the DataFrame | ||
| If a Series or DataFrame is passed, use passed data to draw a table | ||
| Data will be transposed to meet matplotlib’s default layout |
There was a problem hiding this comment.
Can you put full stops . after each sentence? (in this text it looks good, but when it is rendered ti html, the different lines become continuous text)
|
@sinhrks this is correct? |
| table : boolean, Series or DataFrame, default False | ||
| If True, draw a table using the data in the DataFrame. | ||
| If a Series or DataFrame is passed, use passed data to draw a table. | ||
| Data will be transposed to meet matplotlib’s default layout. |
There was a problem hiding this comment.
This transpose is for making plotting internal data to mpl table layout. Thus, transposed only in boolean., not in another DataFrame and Series is passed.
There was a problem hiding this comment.
Great catch thanks! Fixed and PR updated.
|
Thans for this. Only one comment. |
|
@jorisvandenbossche looks ok to me |
|
@mortada Thanks a lot! |
jorisvandenbossche
added a commit
that referenced
this pull request
Jul 9, 2014
DOC: table keyword missing in the docstring for Series.plot() and DataFr...
jorisvandenbossche
added a commit
that referenced
this pull request
Jul 9, 2014
yarikoptic
added a commit
to neurodebian/pandas
that referenced
this pull request
Jul 15, 2014
Version 0.14.1 * tag 'v0.14.1': (21 commits) RLS: v0.14.1 final DOC: clean up 0.14.1 whatsnew file CI: remove sql from 3.3 build entirely DOC: docstring for PeriodIndex SQL: suppress warning for BIGINT with sqlite and sqlalchemy<0.8.2 (GH7433) TST/REG/BUG: Make PeriodIndex work with Series objects DOC: fix problem with part of code example cut off TST: additional network error catching (test_html/test_spam_html) DOC: fix doc build warnings BUG: offset normalize option may not work in addition/subtraction DOC: add release thanks note DOC: fix non-ascii character (from pandas-dev#7698) DOC: table keyword missing in the docstring for Series.plot() and DataFrame.plot() TST/CLN: centralize numpy < 1.7 skips TST: skip on older numpy for (GH7694) BUG: Fix conditional for underlying price in io.data.options. TST: fix dtypes in select_dtypes (GH7434) on windows PERF: better perf on min/max/resample when on indices not containing NaT for DatetimeIndex/PeriodIndex (GH7633) DOC: minor corrections in v0.14.1 FIX: to_sql takes the boolean column as text column BUG: grouped hist raises error with single group ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
...ame.plot()