-
Notifications
You must be signed in to change notification settings - Fork 1
Insertion sort #8
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
base: master
Are you sure you want to change the base?
Conversation
…o bst Merging branch bst.
…o bst sdf building a binary search tree.
…ctures into traversal-bst g g Lines starting with '#' will be ignocquire post_order_trav. red, and an empty message aborts
README.MD
Outdated
|
|
||
| The assignment was to implement a [Weighted Graph](https://codefellows.github.io/sea-python-401d5/assignments/graph_3_weighted.html) | ||
| in Python containing the following methods: | ||
| Insertion Sort takes a list and re-orders it so that the items in the list are arranged lowest to highest. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A more detailed description would be nice -- all your sorting algorithms reorders and sorts the list, but what makes insertion sort unique?
src/insertion_sort.py
Outdated
| """ | ||
|
|
||
|
|
||
| def insertion_sort(isl): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isl is a strange name for this. Why did you pick it? Is there something that you could use that would be a bit more intuitive?
… different orders.
…same size, same contents, different orders.
|
|
||
| def insertion_sort(list): | ||
| """Insertion sort method.""" | ||
| for item in range(len(list)): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
list is a horrible variable name
No description provided.