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

Episode 4: Removing Unneeded Files exercise needs rewrite #848

Open
louiselever opened this issue Sep 27, 2018 · 3 comments
Open

Episode 4: Removing Unneeded Files exercise needs rewrite #848

louiselever opened this issue Sep 27, 2018 · 3 comments
Labels
help wanted Looking for Contributors type:clarification Suggest change to make lesson clearer type:discussion Discussion or feedback about the lesson

Comments

@louiselever
Copy link

In the Pipes and Filters episode, the "Removing Unneeded Files" exercise (part of the closing Nelle's Pipeline story), discusses raw files and the deletion of processed data files.

Suppose you want to delete your processed data files, and only keep your raw files and processing script to save storage. The raw files end in .dat and the processed files end in .txt. Which of the following would remove all the processed data files, and only the processed data files?

This is problematic in the following ways:

  1. Nelle's example data only contains *.txt files (ignoring goo* and do-stats.sh)
  2. Therefore it does not include any raw .dat files, only processed .txt files.
  3. We only get stats-NENE*.txt processed files in the next episode; still no *.dat files.
  4. The exploratory nature of the material invites learners to try commands out.
  5. Use of the word 'Suppose' is insufficient to stop some people trying the commands.
  6. The exercise *.txt files are inadvertently deleted by some attendees.

Suggest, moving to Episode 5 (Loops), keep in the Nelle's Pipeline section, and rewrite to reflect the processed stats-NENE*.txt that are produced and can be removed as unneeded files.

@gcapes
Copy link
Contributor

gcapes commented Nov 16, 2018

All valid points, but I think there are enough exercises in the loops section already, and this exercise is to practice wildcards rather than loops.

It would however be good if the exercise could be tried by learners rather than it being just a thought experiment.

I'm reluctant to create more files in the data-shell directory though, because this lesson is scattered enough as it is. Perhaps it could be made clearer that this is a thought experiment, with instructions on how to create some files with touch, in order to try out the commands?

@gcapes gcapes added help wanted Looking for Contributors type:discussion Discussion or feedback about the lesson type:clarification Suggest change to make lesson clearer labels Jan 16, 2019
@ErinBecker ErinBecker added this to the June 2019 Release milestone May 14, 2019
@deppen8
Copy link
Contributor

deppen8 commented May 6, 2022

Is this addressed by #1233 ?

@gcapes
Copy link
Contributor

gcapes commented May 6, 2022

Is this addressed by #1233 ?

No.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Looking for Contributors type:clarification Suggest change to make lesson clearer type:discussion Discussion or feedback about the lesson
Projects
None yet
Development

No branches or pull requests

4 participants