-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Cookbook text fix #8957
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
Cookbook text fix #8957
Conversation
…L' to give a boolean array.
@stevesimmons Make a new branch off of master, then use |
@stevesimmons yep, good catch ( |
avg_weight = sum(x[x.size == 'S'].weight * 1.5) | ||
avg_weight += sum(x[x.size == 'M'].weight * 1.25) | ||
avg_weight += sum(x[x.size == 'L'].weight) | ||
avg_weight = sum(x[x['size'] == 'S'].weight * 1.5) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe use the same column access notation for weight
as well, to be a bit consistent within one line.
@stevesimmons need to rebase/squash this on master |
@stevesimmons I guess you deleted this branch? want to resubmit? |
Yes, i will redo that branch. I am still learning about git, making clean commits, rebasing branches against upstream head, and have made some mistakes along the way. e.g. i am still not clear if I should worry a push is x commits behind head. On 3 December 2014 22:29:29 GMT+00:00, jreback [email protected] wrote:
Sent from my Android device with K-9 Mail. Please excuse my brevity. |
Closes #8944