Skip to content
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

Васич - всё / {20} #83

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

Conversation

avasich
Copy link

@avasich avasich commented Nov 15, 2015

No description provided.


def merge_sort(seq, start, end):
if end - start > insertion_max:
middle = (start + end) // 2
Copy link
Owner

Choose a reason for hiding this comment

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

fun fact: многие разработчики интерплетаций binary search, merge sort, например в java и С допустили ошибку в этой строке и исправили её только в ~2006. вы тоже.

@MikhailErofeev
Copy link
Owner

1+
2 -, там splay вместо решета :)
3+
4+, но нет IO
5+, но нет IO
6+, но нет IO
7+
8+
9+
10+
11+
12+, коряво написано, мои комменты с сайта не удалены, опечатки
13-, требовался pydoc
14 нету
15+

Спасибо, везде элегантные алгоритмические решения и понятный код, очень приятно читать. К отсутствию IO в сортировках не буду уж придираться.

@avasich
Copy link
Author

avasich commented Jan 11, 2016

Здесь шестая задача отмечена как засчитанная - а в таблице 0 баллов. Кому верить? :)

@MikhailErofeev
Copy link
Owner

прошу прощения, fixed

@avasich
Copy link
Author

avasich commented Mar 26, 2016

Добавил все задачи, кроме последней.

@avasich avasich changed the title Васич - модуль 1 Васич - всё / {20} Mar 26, 2016
self.check_spanning(mst, graph)
self.assertEqual(mst.weight(), 4)
s = {(1, 4), (2, 3), (2, 4)}.symmetric_difference(set(mst._edge))
self.assertTrue(s == {} or s == {(1, 3), (2, 4)})
Copy link
Owner

Choose a reason for hiding this comment

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

Вот здесь не понял

Copy link
Author

Choose a reason for hiding this comment

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

Просто мне надо больше спать. Это такая неочевидная проверка, что вернулся один из двух возможных вариантов: a = (1-4, 2-3, 2-4) или b=(1-3, 1-4, 2-3)
Вместо того, чтобы проверять, что x=a или x=b, зачем-то проверил, что x-a=0 или x-a=b-a.

@MikhailErofeev
Copy link
Owner

2+
13+, но дока плохая
14+
16+
17+
18+
19+, но тест плохой и в реализации есть неоптимальность

@avasich
Copy link
Author

avasich commented Mar 28, 2016

Спасибо.
Действительно, плохой. Переписал, теперь хороший. Неоптимальность тоже ликвидировал.

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