Skip to content

Update cheatsheet #45347

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
Closed

Update cheatsheet #45347

wants to merge 1 commit into from

Conversation

KeeratKG
Copy link

This PR contributes the following items to the cheatsheet:

Copy link
Contributor

@MichaelTiemannOSC MichaelTiemannOSC left a comment

Choose a reason for hiding this comment

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

There are two typos on the first page and what I think is a major error on the last.

  1. In the leftmost Creating DataFrames column, the lower figure mislabels the row labeled 'd' as being labeled 'D'
  2. In the lower-right Subsets box, the explanation that df.iloc[10:20] is wrong. It does not select rows 10-20, but the ten rows labeled 10-19.

As for the major error, it concerns the explanation of apply. When doing apply with axis=1, it means work row-by-row, exposing the columns. So axis=1 should show a horizontal hlghlight, not a vertical highlight. The axis=0 is similarly wrong.

@KeeratKG
Copy link
Author

@MichaelTiemannOSC thank you for the feedback and sorry for the delay in getting back! Will address these asap.

@github-actions
Copy link
Contributor

This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this.

@github-actions github-actions bot added the Stale label Feb 19, 2022
@mroeschke
Copy link
Member

Thanks for the pull request, but this PR appears to be stale. Additionally it appears #43036 must be addressed first before other cheat sheet changes so closing.

Happy to have other pull requests with the label good first issue

@mroeschke mroeschke closed this Feb 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants