Skip to content

Assignment 1 - #108

Closed
A-Ighalo wants to merge 2 commits into
UofT-DSI:mainfrom
A-Ighalo:assignment-1
Closed

Assignment 1#108
A-Ighalo wants to merge 2 commits into
UofT-DSI:mainfrom
A-Ighalo:assignment-1

Conversation

@A-Ighalo

@A-Ighalo A-Ighalo commented Jun 23, 2026

Copy link
Copy Markdown

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

  • I implemented functions for reproducibility
  • I discussed my problem in my terms
  • I discussed the space and time complexity for the function

What did you learn from the changes you have made?

  • I learned how to sort lists with both numbers and zeros
  • I learned to calculate the space and time complexity

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

  • I thought about using a binary search which from class has been said to be a much optimized way of sorting values.

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

  • I faced issues sorting both the numbers and the zeros at the same time. Previously, I used two if statements to loop the numbers and the zeros but focusing on sorting the numbers made my answers more accurate.

How were these changes tested?

  • the changes were made using a function for reporducibility

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

Checklist

  • I can confirm that my changes are working as intended

@github-actions

Copy link
Copy Markdown

Hello, thank you for your contribution. If you are a participant, please close this pull request and open it in your own forked repository instead of here. Please read the instructions on your onboarding Assignment Submission Guide more carefully. If you are not a participant, please give us up to 72 hours to review your PR. Alternatively, you can reach out to us directly to expedite the review process.

@A-Ighalo A-Ighalo closed this Jun 23, 2026
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.

1 participant