Skip to content

Add modified intro python lesson#429

Merged
QuLogic merged 2 commits intoUofTCoders:gh-pagesfrom
aays:add-intro-python
Jan 10, 2019
Merged

Add modified intro python lesson#429
QuLogic merged 2 commits intoUofTCoders:gh-pagesfrom
aays:add-intro-python

Conversation

@aays
Copy link
Member

@aays aays commented Jan 10, 2019

This version of the lesson is largely based on Madeleine's, save for a few minor differences in the first half.

At the end, I've added some brief seaborn, and also use sns.load_dataset to load in a built-in dataset instead of having participants download a file.

@QuLogic
Copy link
Member

QuLogic commented Jan 10, 2019

Checking type(x) == int really should be isinstance(x, int).

@QuLogic
Copy link
Member

QuLogic commented Jan 10, 2019

Also, no spaces around function keyword arguments; this isn't R.

@aays
Copy link
Member Author

aays commented Jan 10, 2019

Thanks - notebook's been updated to reflect the feedback.

(and spaces around arguments removed... curse you, PEP8!)

@QuLogic
Copy link
Member

QuLogic commented Jan 10, 2019

I'm not sure if you are going to mention PEP8, but for docstrings you can also mention numpydoc or google docstring styles. Both are supported by sphinx's napoleon extension and are less verbose than sphinx's default reST. With sphinx, you can make nice HTML docs from your code. (You don't necessarily need to mention all this of course.)

@QuLogic QuLogic merged commit aae9a21 into UofTCoders:gh-pages Jan 10, 2019
@aays aays deleted the add-intro-python branch January 10, 2019 22:59
@QuLogic
Copy link
Member

QuLogic commented Jan 11, 2019

BTW, I tried this out on someone who just installed anaconda and %matplotlib inline didn't seem to be necessary. I think soon we won't need to recommend this.

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.

2 participants