-
Notifications
You must be signed in to change notification settings - Fork 25
Idea for more detailed list comprehension explanation #33
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
The colors make things pretty and clear... but... At the workshop we gave the talk like this:
Do we still do that for some parts of the workshop ? |
Ah... I did not realize we weren't showing the notebook. If that's the case just close this. |
We can always change things... up to a certain time before the show starts ;-) |
Is that a suggestion to replace the highlighting with instructions to copy paste different parts of the loop? I can update this PR if that's the case. |
(Or someone else can, heading to work in a few minutes.) |
Cool. I didn't know you could color notebooks. I think if this section was presented as slides then color coding would work (see slides discussion in #7). I like the idea, but I'm not sure yet whether I want to try rewriting this section to be slides right now. I think we should go with @macro1's colored version so the students can at least see it afterward. |
To address @alaindomissy's comment: only Part 3 is slide-based, and I'll make sure it's as repeat-teachable as the rest. Once I'm done with my slides:
The only reason Part 3 is slides is because it is
This should not affect the live demo nature of Parts 1, 2, and 4. |
@audreyr Great! Thank you! |
976a2ec
to
5407dc7
Compare
Manually merges and closes |
I mentioned in #17 that I might make an image, but this seems to look okay...
http://nbviewer.ipython.org/github/pythonsd/intro-to-python/blob/part-4-list-comprehension/part-4.ipynb