Igor Platonov. Task 7 #18
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
spajic
approved these changes
Sep 3, 2020
| Результат был таков | ||
|  | ||
|
|
||
| Было сделано два запуска. При первом, время выполнения 12 секунд. На втором 11 секунд |
There was a problem hiding this comment.
Это во сколько потоков? Очень странно, что нет профита.
| ### Шаг №4 | ||
|
|
||
| Далее, к сожалению для домашнего задания, но к счастью для проекта, в таких инструментах, как create_default необходимости не было, так как в этом плане спеки были вполне оптимизированы. Также build_stubbed не дал какого-то значительного прироста, да и мест было немного для него | ||
| К сожалению, на данный момент никак не удается преодолеть планку в 10 секунд выполнения всех тестов |
There was a problem hiding this comment.
Кажется вы не весь профит получили от распараллеливания.
Либо оно не заработало, либо тесты очень неравномерно распределились, так что бОльшая часть работы оказалась у одного процесса (ему случайно достались все долгие тесты)
Но в целом 10 секунд и так довольно быстро
| Даже на этом проекте test-prof оказался очень полезным и с помощью него удалось добиться некоторой оптимизации. | ||
| Думаю, на проекте побольше и/или на тестах, которые имеют состояние похуже - он покажет себя во всей красе | ||
|
|
||
| Guard - к сожалению, до сегодняшнего дня я его не применял. А попробовав, не понимаю, как без него раньше жил =) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Добавлен case-study