Skip to content

completed assignment 2#2

Open
Anjishnubose wants to merge 2 commits intomainfrom
assignment-2
Open

completed assignment 2#2
Anjishnubose wants to merge 2 commits intomainfrom
assignment-2

Conversation

@Anjishnubose
Copy link
Copy Markdown
Owner

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Added answers to assignment 2 notebook.

What did you learn from the changes you have made?

Basic data analysis in python from CSV data files.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

No

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

How were these changes tested?

Tested them manually.

A reference to a related issue in your repository (if applicable)

Checklist

  • I can confirm that my changes are working as intended

" # YOUR CODE HERE: Use the readline() or readlines() method to read the .csv file into 'contents'\n",
" \n",
" # YOUR CODE HERE: Iterate through 'contents' using a for loop and print each row for inspection"
" for line in f:\n",
Copy link
Copy Markdown

@kelichiu kelichiu Mar 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The instruction specified use readline() or readlines().
Instead of using a for loop to append each row into contents, we can just do:
contents = f.readlines()

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey,
I corrected this issue and now use readlines() to read the CSV file.
Thanks

Copy link
Copy Markdown

@kelichiu kelichiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incomplete

Code Execution

All code cells execute without errors.

Code Quality

Code is well-organized, concise, and includes necessary comments for clarity.

Data Handling

Data files are correctly handled and processed.

❌ Adherence to Instructions

Did not follows all instructions and requirements as per the assignment. Please see the comments. Use readlines() will lead to best practices.

Specific Criteria

  1. Reading in our files
    Correctly prints out information from the first file.
  2. Summarizing our data
    Correctly defines patient_summary() function. Function processes data as per operation and outputs correctly shaped data (60 entries).
  3. Checking for Errors
    Correctly defines detect_problems() function. Function uses patient_summary() and check_zeros() to identify mean inflammation of 0 accurately.

Overall Assessment

Meets all the general and specific criteria, indicating a strong understanding of the assignment objectives.

Copy link
Copy Markdown

@kelichiu kelichiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Complete

Code Execution

All code cells execute without errors.

Code Quality

Code is well-organized, concise, and includes necessary comments for clarity.

Data Handling

Data files are correctly handled and processed.

Adherence to Instructions

Follows all instructions and requirements as per the assignment.

Specific Criteria

  1. Reading in our files
    Correctly prints out information from the first file.
  2. Summarizing our data
    Correctly defines patient_summary() function. Function processes data as per operation and outputs correctly shaped data (60 entries).
  3. Checking for Errors
    Correctly defines detect_problems() function. Function uses patient_summary() and check_zeros() to identify mean inflammation of 0 accurately.

Overall Assessment

Meets all the general and specific criteria, indicating a strong understanding of the assignment objectives.

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

Successfully merging this pull request may close these issues.

2 participants