Cpu optimization #147
Open
Cpu optimization #147
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
Jun 8, 2024
| Программа поставлялась с тестом. Выполнение этого теста в фидбек-лупе позволяет не допустить изменения логики программы при оптимизации. | ||
|
|
||
| ## Feedback-Loop | ||
| Для того, чтобы иметь возможность быстро проверять гипотезы я выстроила эффективный `feedback-loop`, который позволил мне получать обратную связь по эффективности сделанных изменений за пару минут. |
| - Пробовала оптимизировать ее | ||
| - Проверяла, как изменилась метрика | ||
| - Смотрела, изменился ли отчет профилировщика | ||
| - Если да, сохраняла изменения |
| ### sessions.select для поиска сессий конкретного пользователя | ||
| - Видно на отчете `ruby-prof` (использовала принтер `Flat`) | ||
| - Для оптимизации я добавила хэш `uid_to_sessions`, в который сразу добавляла сессии для конкретных пользователей при парсинге файла | ||
| - Метрика выросла с 0.750 i/s до **5.980 i/s** |
There was a problem hiding this comment.
я бы тут не советовал использовать ips
ips это больше для бенчмарков, когда что-то выполняется реально много раз в секунду
а у нас тут < 1 ips зачастую
| end | ||
| ``` | ||
| - Метрика выросла до **15.200 i/s**, а метрика для файла `data_large.txt` составила **0.033 i/s** | ||
| - Метод `Date.parse` стал занимать меньше процентов в отчете профилировщика |
There was a problem hiding this comment.
с датой можно вообще ничего не делать, это пасхалочка
| - Проблема перестала появляться в отчете профилировщика `ruby-prof` | ||
|
|
||
| ### Обновление массива users_objects с помощью метода + | ||
| - Данная точка роста была обнаружена при помощи профилировщика `rbspy` (Появлялась только при прогоне программы на достаточно большом входном файле, файл `data_large.txt` до сих пор обрабатывался очень долго) |
There was a problem hiding this comment.
лайк за rbspy; его киллер-фича что можно подключиться к работающему процессу и посмотреть где он там застрял
или можно например профилировать процесс пумы и посмотреть на что уходит основное время работы сервера в целом
| Удалось улучшить метрику системы - количество итераций работы программы на файле `data3.txt` - c *0.750 i/s* до *20.820 i/s*, а количетсво итераций работы программы на файле `data_large.txt` до **0.042 i/s** и уложиться в заданный бюджет. | ||
|
|
||
| ## Защита от регрессии производительности | ||
| Для защиты от потери достигнутого прогресса при дальнейших изменениях программы я добавила perfomance-тест `spec/task-1_spec.rb` |
|
|
||
| describe 'Perfomance' do | ||
| describe 'task 1' do | ||
| it 'works under 30 seconds' do |
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.
No description provided.