-
-
Notifications
You must be signed in to change notification settings - Fork 780
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
05-lists.md: Added missing explanation bit. #831
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @Fnyasimi , thank you for taking the time to bring up the possibility of confusion in this exercise.
_episodes/05-lists.md
Outdated
@@ -441,7 +441,7 @@ last: 4 | |||
> {: .output} | |||
> | |||
> Use the step size argument to create a new string | |||
> that contains only every other character in the string | |||
> that contains only every other second character in the string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For me, the original explanation is more clear than with the addition of the word second
because when I read every other second character
I think perhaps it would be asking for the second character, fourth character, sixth character, etc. instead of the first, third, fifth, etc. that the expected output shows with I notpssgre ntesae
. Perhaps if you and others find the exercise explanation "every other character in the string" to be unclear, we could work on making it more clear that the output should look like I notpssgre ntesae
by retaining the first character, third character, fifth character...in some further way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ldko for the review feedback I think the second explanation of making it clear how the output should look like will be more understandable to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Fnyasimi, would you like to modify this PR to do that or would you like me to commit a suggestion on your branch and you can see what you think?
I am thinking the text could change to something like:
> Similarly, we can take non-continuous slices of strings.
> Use the step size argument to create a new string
> that contains only every other character in the string
> "In an octopus's garden in the shade". Start with
> creating a variable to hold the string:
>
> ~~~
> beatles = "In an octopus's garden in the shade"
> ~~~
> {: .language-python}
>
> What slice of `beatles` will produce the
> following output (i.e., the first character, third
> character, and every other character through the end
> of the string)?
> ~~~
> I notpssgre ntesae
> ~~~
> {: .output}
Do you think this text makes the exercise more clear?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ldko This is more clear and understandable of what is expected to be done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you like to make these changes to the explanation in another commit @Fnyasimi ?
@ldko Kindly review the new changes I have made. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That works for me. Thank you for making the change @Fnyasimi ! 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks great! Thanks, @Fnyasimi!
* Added missing explanation. * Update to make the question clear
I have come across missing information in this line
Use the step size argument to create a new string that contains only every other character in the string “In an octopus’s garden in the shade”
In the new correction I have added an the explanation to guide the learner in on what to do to get the desired output.
New sentence
Use the step size argument to create a new string that contains only every other
second
character in the string “In an octopus’s garden in the shade”