Skip to content

DOC iteritems docstring update and examples #22658

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.

Sign up for GitHub

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

Merged
merged 12 commits into from
Sep 27, 2018
Merged

DOC iteritems docstring update and examples #22658

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
0521552
DOC
Ecboxer Sep 2, 2018
5447a1f
Continuation lines start with ... and are aligned, reverted descripti…
Ecboxer Sep 11, 2018
8ccd554
Removed trailing space and added Returns/Yields section
Ecboxer Sep 12, 2018
fcc27e8
Added extended summary
Ecboxer Sep 12, 2018
6dad21c
Edits to long description, split Yields section into label and conten…
Ecboxer Sep 23, 2018
25da7f8
Merge remote-tracking branch 'upstream/master' into shiny-new-feature
Ecboxer Sep 23, 2018
30026a4
Changed pop to population and reformatted example
Ecboxer Sep 25, 2018
5110b7c
Changed pop to population in example output
Ecboxer Sep 25, 2018
76243b3
Changed docstring to string literal and removed escape chars
Ecboxer Sep 25, 2018
1b52a08
Removed end="\n"
Ecboxer Sep 25, 2018
618318c
Shortened example, fit long desc to char limit, changed Yields to des…
Ecboxer Sep 25, 2018
d8e5370
Returned to two part Yields section, label and contents
Ecboxer Sep 25, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 20 additions & 1 deletion pandas/core/frame.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -726,13 +726,32 @@ def style(self):

def iteritems(self):
"""
Iterator over (column name, Series) pairs.
Copy link
Member

Choose a reason for hiding this comment

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

Technically this returns an Iterator (or more specifically a Generator). So I would leave this as Iterator or change this to Generator.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought to make the change to 'Iterate over DataFrame ... as (..., Series) pairs to stay within the style of the iterrows and itertuples functions, but I can revert back to 'Iterator over (column name, Series) pairs.

Iterate over DataFrame columns as (column name, Series) pairs.

See also
--------
iterrows : Iterate over DataFrame rows as (index, Series) pairs.
itertuples : Iterate over DataFrame rows as namedtuples of the values.
Copy link
Member

Choose a reason for hiding this comment

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

A in See Also should be capitalized.

Use DataFrame.iterrows instead of iterrows and same for itertuples.


Examples
--------

>>> df = pd.DataFrame({'col1': [1, 2], 'col2': [0.1, 0.2]},
index=['a', 'b'])
Copy link
Contributor

Choose a reason for hiding this comment

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

Continuation lines need to start with ... (right below the >>>)

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, make sure the i in index is right below the opening { on the line above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gotcha, I will make the changes.

>>> df
col1 col2
a 1 0.1
b 2 0.2
>>> for col in df.iteritems():
... print(col)
Copy link
Member

Choose a reason for hiding this comment

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

I think something like next would make things easier to understand:

for label, content in df.iteritems():
    print('label:', label)
    print('content:' content)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the edits! I rephrased the long description, changed the Yields section to cover label and content, prepended DataFrame. to the See Also examples, and reworked the Examples section to hopefully be more engaging.

Copy link
Member

Choose a reason for hiding this comment

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

did you push the changes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I got my git commands mixed up sheepishly, but it's pushed now

...
('col1', a 1
b 2
Name: col1, dtype: int64)
('col2', a 0.1
b 0.2
Name: col2, dtype: float64)

"""
if self.columns.is_unique and hasattr(self, '_item_cache'):
for k in self.columns:
Expand Down