Skip to content

Conversation

@akokai
Copy link
Contributor

@akokai akokai commented Sep 23, 2016

Only thought of a few really minor things so far.

  • Lists: Can't seem to access the <function>? help mechanism in Jupyter for methods. Running, e.g., ''.join? just returned "Object not found" or something like that. So instead, suggesting to look up the docs on join and clarifying that it's a string method, not a list method.
  • Loops: The use of loops + conditionals to "evolve" a variable iteratively needed a bit of contextualization & explanation. People were a bit mystified why anyone would do this. I suggested population modeling (ecology, etc.) as one possible reason.

@akokai
Copy link
Contributor Author

akokai commented Sep 23, 2016

One more thing:

  • Lists: Added explanation of the behavior of variables created by slicing a list, when the original list gets changed. This is counterintuitive because of the mutability of lists, etc. Maybe someone should review my explanation to make sure it's not confusing.

@henchc
Copy link
Contributor

henchc commented Sep 23, 2016

in jupyter you have to type the datatype to get the help:

str.join?
str.split?
list.append?
list.extend?

did that work for you? but going to the docs works too, if it works, can you add the str.join? back to the notebook as a second reference?

all the other changes look great, we'll let @alexestes and @natalieahn give input, then go ahead and merge

@alexestes
Copy link
Contributor

Looks good to me!

On Fri, Sep 23, 2016 at 3:26 PM, Christopher Hench <notifications@github.com

wrote:

in jupyter you have to type the datatype to get the help:

str.join?
str.split?
list.append?
list.extend?

did that work for you? but going to the docs works too, if it works, can
you add the str.join? back to the notebook as a second reference?

all the other changes look great, we'll let @alexestes
https://github.com/alexestes and @natalieahn
https://github.com/natalieahn give input, then go ahead and merge


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#3 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUF6zEbeuuJCwwXomOJsBrHoMTURrlKzks5qtFIdgaJpZM4KFNGc
.

@natalieahn
Copy link

Changes look good to me too!

@akokai
Copy link
Contributor Author

akokai commented Sep 24, 2016

Hah, TIL that you need to say str.join? :) Thanks, it works. I reinstated the Jupyter help thing as the primary course of action, with a link to the Python docs just to show people that all of this is also on a website you can access outside of Jupyter.

I'm still not fully satisfied with the 'population' loops/conditionals example – I have a feeling we can find a more intuitive and straightforward example of that pattern – but I'll leave it for now.

I think we can merge this whenever the intensive series is done.

@henchc henchc merged commit 914e0a2 into master Sep 28, 2016
@pssachdeva pssachdeva deleted the daytwofix branch December 20, 2021 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants