Update official docs #491
Update official docs #491
Conversation
| def do_laundry(clothes): | ||
| wet_clothes = wash(clothes, coins) | ||
| dry_clothes = dry(wet_clothes, coins) | ||
| wet_clothes = wash(clothes) | ||
| dry_clothes = dry(wet_clothes) | ||
| return fold(dry_clothes) |
There was a problem hiding this comment.
I feel the presence of coins is somewhat antithetical to the principal of function purity as described in the docs.
| >>> double = partial(mul, 2) # Partial evaluation | ||
| >>> doubled = double(2) # Currying | ||
| >>> doubled = double(5) # Currying |
There was a problem hiding this comment.
I feel the choice of passing in 2 might be slightly confusing for some people given the function itself is called double.
| @@ -38,7 +38,7 @@ These functions correspond to the SQL commands ``SELECT`` and ``WHERE``. | |||
| ... map(get([1, 2])), | |||
| ... list) | |||
|
|
|||
| Note: this uses the `curried`_ versions of ``map`` and ``filter``. | |||
There was a problem hiding this comment.
Removing cross linking because the page is not rendering correctly, as seen below.
| @@ -89,7 +89,7 @@ groups. | |||
| 'M': [(2, 'Bob', 200, 'M'), (3, 'Charlie', 150, 'M'), (4, 'Dennis', 50, 'M')]} | |||
|
|
|||
| >>> valmap(compose(sum, pluck(2)), | |||
| ... _) | |||
| ... _) # The underscore captures results from the previous prompt | |||
There was a problem hiding this comment.
This might not be obvious to everyone.
| @@ -2,7 +2,7 @@ Tips and Tricks | |||
| =============== | |||
|
|
|||
| Toolz functions can be combined to make functions that, while common, aren't | |||
| a part of toolz's standard library. This section presents | |||
| a part of toolz's standard offerings. This section presents | |||
There was a problem hiding this comment.
To avoid confusion with the python standard library.
|
This looks great! Thanks so much @zzhengnan! Sorry for the delay. I'm striving to be more responsive towards |
|
@eriknw Thanks for the review! |
This PR contains
I'll push the generated docs once the RST changes are considered LGTM.