task_1 #131
task_1 #131
Conversation
| - ruby-prof в формате flat показал, что следующая точка роста Array#all? | ||
| - получение списка уникальных браузеров через map и uniq: | ||
| `unique_browsers = sessions.map { |session| session['browser'] }.uniq` | ||
| - время выполнения сократилось с 1,03 s до 0,37 s |
There was a problem hiding this comment.
Стало слишком маленькое время. Тут было бы хорошо увеличить размер файла, чтобы программа успевала покрутиться секунд 5 в основном цикле
| Я решил исправить эту проблему, оптимизировав эту программу. | ||
|
|
||
| ## Формирование метрики | ||
| Для того, чтобы понимать, дают ли мои изменения положительный эффект на быстродействие программы я придумал использовать такую метрику: время обработки файла на 10 000 строк в секундах |
There was a problem hiding this comment.
Нам такая метрика не совсем подходит на всём протяжении этой работы.
Мы можем зафиксировать такую метрику на какой-то момент, потом взять файл побольше, потом ещё побольше. В таком случае нам та или иная метрика позволяет оценить эффект оптимизации, которую мы делаем на данном шаге
В идеале нас интересует время выполнения на полном объёме, но так как слишком долго ждать, мы ищем другие (временные) метрики.
|
|
||
| ## Результаты | ||
| В результате проделанной оптимизации наконец удалось обработать файл с данными. | ||
| Удалось улучшить метрику системы и уложиться в заданный бюджет. После проделанной работы обработка файла занимает ~40 s. |
There was a problem hiding this comment.
Пойдёт, мб во втором задании удастся в 30 секунд уложиться
|
|
||
| report['uniqueBrowsersCount'] = uniqueBrowsers.count | ||
|
|
||
| unique_browsers = sessions.map { |session| session['browser'].upcase }.uniq |
There was a problem hiding this comment.
можно их в Set напихивать, в итоге получим только уникальные и очень эффективно
No description provided.