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

Кабанова №7,14,16 #19

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

Кабанова №7,14,16 #19

wants to merge 31 commits into from

Conversation

catr1ne55
Copy link
Contributor

No description provided.

@catr1ne55 catr1ne55 changed the title Task 2 completed Кабанова №2 Sep 17, 2015
@catr1ne55 catr1ne55 changed the title Кабанова №2 Кабанова №2,3 Sep 18, 2015
@MikhailErofeev
Copy link
Owner

2 ok
3, для printf '6\n1 2 3' | python /Users/erofeev/projects/students/a3200-2015-algs/coins .py
получаю ошибку:
line 13, in methods
a[0][j] = 1
IndexError: list assignment index out of range

Похвально, что вы пытаетесь реализовать задачу через динамическое программирование, но она в такой (упрощённой) формулировке имеет гораздо более простое решение. Если отчаятесь делать через динамику, то можете придумать его. Но динамика это очень круто, такое решение в первом дз не ожидалось 👍

a[i, j] = a[i - 1][j] + a[i][j - coins[i]]
return a[len(coins)][quantum]

print 'The result is: ', methods(n, m)
Copy link
Owner

Choose a reason for hiding this comment

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

По спецификации задачи ожидается, что будет возвращено число без текста. Это важно, потому что я могу проверять, например, автотестами

@catr1ne55 catr1ne55 changed the title Кабанова №2,3 Кабанова №2,3,4 Sep 19, 2015
@catr1ne55 catr1ne55 changed the title Кабанова №2,3,4 Кабанова №2,3,4,5 Sep 25, 2015
line = sys.stdin.readline()
list_of_elements = [int(k) for k in line.split(' ')]

stop = int(input('Введите значение для k = '))
Copy link
Owner

Choose a reason for hiding this comment

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

Значение предлагается задать вам самостоятельно константой. Ожидается, что оно будет идеальным, и его не нужно будет конфигурировать при запуске задачи

@MikhailErofeev
Copy link
Owner

5 ок, в 4 прошу подставить вашу константу.
Не могу засчитать все работы в PR, пока все не приняты (в 3 баг). Чтобы была 1 задача на PR, можно, например, разрабатывать их в отдельных ветках своего репозитория и отправлять на PR только нужную ветку.

@catr1ne55 catr1ne55 changed the title Кабанова №2,3,4,5 Кабанова №2,3,4,5,6 Oct 2, 2015
@catr1ne55 catr1ne55 changed the title Кабанова №2,3,4,5,6 Кабанова №2,3,4,5,6,8,11 Nov 12, 2015
@catr1ne55 catr1ne55 changed the title Кабанова №2,3,4,5,6,8,11 Кабанова №2,3,4,5,6,8,10,11,12 Nov 13, 2015
from histogram import max_square


class TestMaxSquare(unittest.TestCase):
Copy link
Owner

Choose a reason for hiding this comment

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

тесты похоже не запускались, первые 3 падают
но алгоритм правильный

@MikhailErofeev
Copy link
Owner

8+, обратите внимание на комментарии
7-, O(n^3) получилось
11+, плохо, что вы не запускали тесты, сейчас всё получилось, но разрабатывать с ними проще
12-, проблемы с выделением памяти, тривиальные ошибки, некрасивый большой класс вместо изящного yield и опять поломанные тесты
16-, падают тесты, дальше смотреть не стал

v.path_to = u.path_to + v.path


graph = WeightedGraph()
Copy link
Owner

Choose a reason for hiding this comment

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

это должно быть в тестах

@catr1ne55 catr1ne55 changed the title Кабанова №2,3,4,5,6,8,10,11,12,16 Кабанова №6,10,12,13,16 Jan 27, 2016
@catr1ne55 catr1ne55 changed the title Кабанова №6,10,12,13,16 Кабанова №6,10,12,13,16,18 Jan 27, 2016
@catr1ne55 catr1ne55 changed the title Кабанова №6,10,12,13,16,18 Кабанова №6,10,12,13,16,18,19 Jan 29, 2016
@catr1ne55
Copy link
Contributor Author

Ссылка на 20 задачу: http://codeforces.com/contest/525/submission/15680362

@catr1ne55 catr1ne55 changed the title Кабанова №6,10,12,13,16,18,19 Кабанова №6,10,12,13,16,18,19,20 Jan 30, 2016
@MikhailErofeev
Copy link
Owner

10+
13+
18+
12+, посмотрите замечание
19+ необычные и очень просто решение, где-то подвох в асимптотике, но я его не нашёл :)
20+

@catr1ne55 catr1ne55 changed the title Кабанова №6,10,12,13,16,18,19,20 Кабанова №14,16 Jan 31, 2016
@catr1ne55 catr1ne55 changed the title Кабанова №14,16 Кабанова №7,14,16 Feb 1, 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