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

[FIX] the list of lists image #427

Open
wants to merge 3 commits into
base: gh-pages
from

Conversation

Projects
None yet
3 participants
@Shotgunosine

Shotgunosine commented Nov 9, 2017

The image in python's list tutorial isn't a great representation of a list of lists. The physical representation of a list isn't consistent (pepper shaker, pepper packet). I made a new image using a table as the physical representation of a list, so a list of lists is three tables stacked on a table.

@valentina-s

This comment has been minimized.

Contributor

valentina-s commented Nov 20, 2017

Thank you for suggesting to improve the representation. I have been thinking that the last picture of the pepper example might have been better if it were just one pepper crumb.

Could you also add a short caption for your explanation? I first did not see the table underneath so I was confused why the [x[0]] was two crossed tables. I even think that maybe skipping [x[0]] will make things more clear for the beginner. I will label the PR for discussion so we hear what others think about this representation.

@maxim-belkin maxim-belkin force-pushed the swcarpentry:gh-pages branch from 4753644 to 90848ea Oct 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment