Skip to content

Conversation

@Nifacy
Copy link

@Nifacy Nifacy commented Nov 19, 2025

Выполнена лабораторная работа №1:

  • Проведен анализ бейзлайн решения
  • Были выполнены оптимизации бейзлайна для получения более точного решения
  • Был проведен итоговый анализ и сравнение полученных решений

P. S.: Так как ветки lab1 в исходном репозитории нет, то в Pull Request выбрал в качестве target branch ветку master

P. S.: Так как milestone-ов в исходном репозитории нет, то их не указывал

Copy link
Contributor

@pivovarov-mai pivovarov-mai left a comment

Choose a reason for hiding this comment

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

Написал пару комментариев. Работа зачтена, можете ее пушить в репозиторий.

"\n",
"with open(\"datasets/lab1-04.csv\", newline='', encoding='utf-8') as f:\n",
" reader = csv.reader(f)\n",
" for row in reader:\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Такие данные можно загружать не построчно, а сразу скопом. С помощью pandas.

Copy link
Author

Choose a reason for hiding this comment

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

Да, но в условии задачи не было явно указано использовать pandas :) Учитывая условия 2-й лабораторной работы подумал, что тут его использовать нельзя.

Copy link
Author

Choose a reason for hiding this comment

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

Поправил. Теперь для загрузки датасета используется pandas

Comment on lines +235 to +236
"plt.xlabel(\"$x$\")\n",
"plt.ylabel(\"$y = f(x)$\")\n",
Copy link
Contributor

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.

Отдельное задание настроек делал намеренно с целью, чтобы блоки кода были идемпотентны (в идеале результат каждой ячейки должен быть одинаковым вне зависимости от порядка запуска). Поэтому попытался глобальное состояние свести к минимуму. Ну и по этой причине предлагаю так оставить.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants