Skip to content

03-index-slice-subset.md: improve wording#340

Merged
wrightaprilm merged 2 commits intodatacarpentry:gh-pagesfrom
welozano:gh-pages
Nov 27, 2018
Merged

03-index-slice-subset.md: improve wording#340
wrightaprilm merged 2 commits intodatacarpentry:gh-pagesfrom
welozano:gh-pages

Conversation

@welozano
Copy link
Copy Markdown
Contributor

The sentence refers to x > 5, I think there is an extra "not"

Wil

The  sentence refers to x > 5, I think there is an extra "not"
@maxim-belkin maxim-belkin self-assigned this Nov 24, 2018
@maxim-belkin maxim-belkin added the type:typo text Typo in text for the lesson, label Nov 24, 2018
@maxim-belkin maxim-belkin changed the title Extra not 03-index-slice-subset.md: improve wording Nov 24, 2018
@maxim-belkin
Copy link
Copy Markdown
Contributor

Thanks, @welozano! good catch! We can merge this as is. However, I'd like to suggest that we spend a bit more time and rewrite this paragraph a bit as it's not super well written.

My suggestion (feel free to improve):

When we ask Python whether x is greater than 5, it returns False. This is Python's way to say "No". Indeed, the value of x is 5, and 5 is not greater than 5.

@wrightaprilm, please chime in. :)

@wrightaprilm
Copy link
Copy Markdown
Contributor

I like your replacement text, Maxim. If you're willing to add it in, @welozano, we would very much appreciate it!

@welozano
Copy link
Copy Markdown
Contributor Author

Hi, the suggestion looks pretty good. I just added as @maxim-belkin suggested.
Wil

@wrightaprilm
Copy link
Copy Markdown
Contributor

Great, thanks @welozano!

@wrightaprilm wrightaprilm merged commit 300513c into datacarpentry:gh-pages Nov 27, 2018
idwaker pushed a commit to pywaker/python-ecology-lesson that referenced this pull request Feb 5, 2019
zkamvar pushed a commit that referenced this pull request May 8, 2023
03-index-slice-subset.md: improve wording
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:typo text Typo in text for the lesson,

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants