Conversation
anjali-deshpande-hub
left a comment
There was a problem hiding this comment.
Please make the following changes:
-
Replace the hardcoded absolute paths to relative paths so that a Learning support can run the code successfully as well. Change
"/Users/zeinabsadr/Desktop/assignment_2_data/inflammation_01.csv"to../../assignment_2_data/inflammation_01.csv" -
In Question 1, the requirement is to use readlines/readline method to read the csv file into a variable (which you have done properly) and use the variable in a for loop to print each row. The for loop is missing. Could you please add that part?
|
Hello Anjali, I updated my file, is that correct ? Thanks. |
|
I am not seeing the changes in assignment_2.ipynb. Perhaps, you forgot to push your changes to github. |
|
Hello, |
|
By mistake, I merged and closed that. I think you can see my changes for Assignment 2 now. |
|
I am not sure which PR you want me to review now. Please revert this PR. Follow the steps outlined here to revert: https://github.com/UofT-DSI/onboarding/blob/main/onboarding_documents/submissions.md#handling-accidental-merges |
|
|
|
Hi, is it fix? |
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist