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

Missing solutions to, context for, and files required by exercises #518

Closed
gcapes opened this issue Feb 17, 2017 · 5 comments
Closed

Missing solutions to, context for, and files required by exercises #518

gcapes opened this issue Feb 17, 2017 · 5 comments

Comments

@gcapes
Copy link
Contributor

gcapes commented Feb 17, 2017

Many of the exercises don't have solutions. Some don't need solutions because learners can run the commands and see which is the correct answer. In some of the exercises, the commands can't be tested because the data files don't exist e.g. 07-find, "matching and subtracting" - there are no *ose.dat files

The -v flag to grep inverts pattern matching, so that only lines which do not match the pattern are printed. Given that, which of the following commands will find all files in /data whose names end in ose.dat (e.g., sucrose.dat or maltose.dat), but do not contain the word temp?

find /data -name '*.dat' | grep ose | grep -v temp
find /data -name ose.dat | grep -v temp
grep -v "temp" $(find /data -name '*ose.dat')
None of the above.

Sometimes it isn't clear which directory the question refers to.

I plan to submit a series of PRs to address these issues, using existing files as much as possible. Is this of interest to others?

@shwina
Copy link

shwina commented Feb 17, 2017

@gcapes - yes, of course! PRs addressing these issues are more than welcome.

@gcapes
Copy link
Contributor Author

gcapes commented Feb 21, 2017

@shwina
See #520.
A bit of context for the latter part of this PR: I think it's very helpful for learners to be able to try the commands, rather than only having a mental exercise e.g. 05-loops/variables in loops

Suppose that ls initially displays...

It is for this reason that I'll be changing the exercises on this episode to use existing files, rather than providing solutions for the exercises as they currently stand.
Cheers

@gcapes
Copy link
Contributor Author

gcapes commented Feb 28, 2017

@shwina
I've now finished submitting PRs to fix this issue:

I only had to add files for #528.

I see that #524 adds to #521, but #523 is a duplicate of one of the commits in #522.
Cheers

@shwina
Copy link

shwina commented Feb 28, 2017

Thanks for all your contributions and for this summary @gcapes! I'll begin reviewing ASAP.

@gcapes
Copy link
Contributor Author

gcapes commented Feb 28, 2017

Great, thanks!

@shwina shwina closed this as completed in 3d186a8 Feb 28, 2017
shwina pushed a commit that referenced this issue Mar 7, 2017
shwina pushed a commit that referenced this issue Mar 7, 2017
shwina pushed a commit that referenced this issue Apr 14, 2017
shwina pushed a commit that referenced this issue Apr 19, 2017
zkamvar pushed a commit that referenced this issue May 2, 2023
zkamvar pushed a commit that referenced this issue May 2, 2023
zkamvar pushed a commit that referenced this issue May 2, 2023
zkamvar pushed a commit that referenced this issue May 2, 2023
zkamvar pushed a commit that referenced this issue May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants