-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
DOC:Remove supression from ipython directive to make object visible #11443
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
Conversation
That'd be fine, or maybe put a note before each saying something like "First some setup" and "with the setup out of the way...". Either way, it'll be an improvement 👍 |
01e34cd
to
c81dc5b
Compare
Added some 'verbal sugar' on either side of the Ipython directive blocks. Hope that's ok. |
@@ -268,6 +269,8 @@ natural to group by one of the levels of the hierarchy. | |||
index = pd.MultiIndex.from_tuples(tuples, names=['first', 'second']) | |||
s = pd.Series(np.random.randn(8), index=index) | |||
|
|||
We can then group by one of the levels in ``s``. | |||
|
|||
.. ipython:: python | |||
|
|||
s |
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.
I would put this s
at the end of the previous block
Looks good! I just put some comments, but that is actually more about the current content than your change :-) |
@Winterflower Do you have some time to update this? |
yep, @jorisvandenbossche will do tonight! |
@Winterflower ping when pushed & green |
DOC: Some minor additional fixes Added some corrections
c81dc5b
to
eaabe81
Compare
@jreback pushed and green! |
DOC:Remove supression from ipython directive to make object visible
thanks! |
Fix for #11223
@TomAugspurger should these two Ipython blocks be merged into one?
(this is what it looks like at the mo)
