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

Update 01-numpy.md on dot notation. #310

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

Conversation

Projects
None yet
5 participants
@rozmanov

rozmanov commented Sep 16, 2016

The use of the dot notation in accessing properties and methods of an object are not clearly stated in the lesson. I added a diagram and a brief discussion to address this issue.

This is the fist step of my graduation procedure.

Update 01-numpy.md on dot notation.
The use of the dot notation in accessing properties and methods of an
object are not clearly stated in the lesson.
I added a diagram and a brief discussion to address this issue.
@tbekolay

This comment has been minimized.

Contributor

tbekolay commented Sep 28, 2016

This PR contains the diagram, but I don't see the brief discussion, perhaps it wasn't git added?

Anyhow, I worry about the cognitive load involved in introducing the idea of "objects"... we try to gloss over this topic deliberately as it's quite deep and we have too much material as is. What do you suggest that we remove to make room for a discussion of objects?

Update 01-numpy.md on dot notation (text added);
The use of the dot notation in accessing properties and methods of an
object are not clearly stated in the lesson.
I added a diagram and a brief discussion to address this issue.
@rozmanov

This comment has been minimized.

rozmanov commented Sep 28, 2016

Yes, I did not add the updated text. Sorry about it. I updated the text and pushed it to my repository. Should it be enough for you to see the changes? The discussion is really brief, but in my opinion it is important to touch upon the issue. At least this should be better than some partisan introduction of an idea, I guess.

rgaiacs pushed a commit to rgaiacs/swc-python-novice-inflammation that referenced this pull request May 6, 2017

@valentina-s

This comment has been minimized.

Contributor

valentina-s commented Oct 23, 2017

I find this concept too complicated for the beginning of the lesson, and I don't think we should make the lesson any longer. Maybe putting the example in a callout or an optional exercise will be more appropriate? The exercise can have a simple question: name one property/method of a numpy array: eg shape, max().

maxim-belkin pushed a commit that referenced this pull request Sep 11, 2018

Replace page.root (#310)
* replace page.root with 'base' variable that knows path level

* deal with case of trailing / in URL

* use link to figure out path for logos

* simplify path building for favicons

* baseurl isn't used in our template so it can removed
* use 'base' variable for path root instead of unecessary http request to
  site.url

* replace page.root with 'base' variable that knows path level

* deal with case of trailing / in URL

* use link to figure out path for logos

* simplify path building for favicons

* baseurl isn't used in our template so it can removed
* use 'base' variable for path root instead of unecessary http request to
  site.url

* rename variable, consolidate comments

* rename variable in template files

@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