-
Notifications
You must be signed in to change notification settings - Fork 15
Suggestions for Ch 1 - Pandas intro #37
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
Comments
@joelostblom "Better explanation of alt.X and alt.Y, take from 531 lecture notes." can you paste that here pls? |
Actually, just file a new issue for it -- what is there right now in Chapter 1 is good enough for running the course. We can (and should) make enhancements later. |
The separate issue threads I think again are "nice to have"s, but not mandatory for the first run of the course, so checking them all off |
Yes I can open a new issue for that! I think it's useful if we open new issues or leave unchecked all the things we don't implement just so that we can keep track of it and decide if we want it later. |
That's what I've been doing -- "checked off" means either handled in the upcoming PR or spun out into its own issue for later. |
These are all suggestions, some might seem more harshly written, but that is just to be succinct and write quickly.
.
-
instead of+
is a more natural example for non-working var names....
that could use an explanation. And the number of rows and columns are printed at the bottom, not top..query
or[]
? The former has some advantages in clearer chaining syntax, but is not as well documented..
, it is the open paren (which is preferred over "")?
also works in place ofhelp
in ipython, I don't think printing the docstring is worth teaching.The text was updated successfully, but these errors were encountered: