You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hello Sofia, this is the review I have for your first Project.
Overall, I have no problem unzip your file and get the text in order to run the code. Thank you for arranging it properly so that I can easily begin the review process.
For the first problem set about word counts, I find the way you used “ | “to combine to be intelligent, because when I did the problem, I used separate code to process and I find yours to be much more efficient. One more thing I notice is that for the Part B.; because the exact pattern we are looking for is “Rom. ”, which contains a dot and a space after Rom and Jul, I think it would be more precise if you consider adding these criteria into your searching pattern.
As for the second problem- the Capital Bikeshare problem, my approach is exact the same as yours in both part A and part B.
For the third problem, my method is again almost the same as yours. One thing I learned from viewing your simplefilter python code is the usage of adding space to the strip function that help split words. I would consider using this method next time I encounter this type of problem.
For the extra credit, the way I prepare the text file is the same as yours. However, I would suggest using GNU parallel and append it into a new txt file before sort.
The experience of reviewing your code went on very smoothly. Although I wish I could see more explanations about why you select the method you used, it is still very easy for me to follow your logic and get the result from your code. Thank you for arranging it clearly and I really appreciate the great work you have!
Szu
The text was updated successfully, but these errors were encountered:
Hello Sofia, this is the review I have for your first Project.
Overall, I have no problem unzip your file and get the text in order to run the code. Thank you for arranging it properly so that I can easily begin the review process.
For the first problem set about word counts, I find the way you used “ | “to combine to be intelligent, because when I did the problem, I used separate code to process and I find yours to be much more efficient. One more thing I notice is that for the Part B.; because the exact pattern we are looking for is “Rom. ”, which contains a dot and a space after Rom and Jul, I think it would be more precise if you consider adding these criteria into your searching pattern.
As for the second problem- the Capital Bikeshare problem, my approach is exact the same as yours in both part A and part B.
For the third problem, my method is again almost the same as yours. One thing I learned from viewing your simplefilter python code is the usage of adding space to the strip function that help split words. I would consider using this method next time I encounter this type of problem.
For the extra credit, the way I prepare the text file is the same as yours. However, I would suggest using GNU parallel and append it into a new txt file before sort.
The experience of reviewing your code went on very smoothly. Although I wish I could see more explanations about why you select the method you used, it is still very easy for me to follow your logic and get the result from your code. Thank you for arranging it clearly and I really appreciate the great work you have!
Szu
The text was updated successfully, but these errors were encountered: