Skip to content
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

Feedback for Episode 6. Working with free text (sort -nr and carriage return) #183

Open
bellaratmelia opened this issue Jan 10, 2022 · 0 comments
Labels
type:clarification Suggest change for make lesson clearer

Comments

@bellaratmelia
Copy link

I have two feedback/suggestions for Episode 6. working with free text:

First feedback:

I think it will be great to mention that the command sort -nr will reorder things in reverse order. At the moment, the explanation only said:

The fourth and final part sorts the text again by the counts of duplicates generated in step three.

So perhaps the following sentence can be added:

The -n part of the flag indicates that it's sorting numerically, and the r part of the flag means it's sorting in reverse (descending) order.

Second feedback:

I think it will be useful to put a brief explanation on what is a "carriage return", how it is different from newline, and why it needs to be removed from the text. I'm specifically referring to this text in the episode:

This uses the translate command and a special syntax to remove all punctuation ([:punct:]) and carriage returns (\r).

I don't know if it's just me but I have never heard about that term before this lesson. I tried searching around a little bit but I'm still not quite sure about why it needs to be removed from the text during cleaning.

Thank you!

@kaitlinnewson kaitlinnewson added the type:clarification Suggest change for make lesson clearer label Sep 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:clarification Suggest change for make lesson clearer
Projects
None yet
Development

No branches or pull requests

2 participants