Skip to content

Исправление: Налитов Денис. Технология SEQ. Поиск кратчайших путей из одной вершины (алгоритм Дейкстры). Вариант 21#1130

Merged
allnes merged 1 commit into
learning-process:masterfrom
a11ydenis:nalitov_d_dijkstras_algorithm_seq_fix
May 17, 2026

Conversation

@a11ydenis
Copy link
Copy Markdown
Contributor

Данные об исходной задаче

Задача: Поиск кратчайших путей из одной вершины (алгоритм Дейкстры)
Вариант: 21
Технология: SEQ
Директория студента: nalitov_d_dijkstras_algorithm
Ссылка на исходный PR / коммит / issue: #135
Полное описание исходной задачи: Реализация алгоритма Дейкстры для поиска кратчайших путей из заданной вершины-источника во взвешенном ориентированном графе с неотрицательными весами рёбер. Результат — сумма кратчайших расстояний от источника до всех достижимых вершин.

Детали исправления

Проблема: Исходная SEQ-реализация использовала heap-алгоритм Дейкстры (std::priority_queue) со сложностью O((n+m) log n). Параллельные версии реализованы по квадратичной схеме O(n²). При разнородном подходе сравнение ускорения теряет смысл: распараллеленная программа всегда будет отставать от реализации на эффективном алгоритме.

Суть исправления: Чтобы обеспечить корректное измерение ускорения при параллелизации, последовательная версия переведена на квадратичный алгоритм.

Чек-лист

  • Статус CI: Все CI-задачи (сборка, тесты, генерация отчёта) успешно проходят на моей ветке в моем форке
  • Директория и именование задачи: Я не переименовывал существующую директорию задачи и не добавлял лишние директории задач
  • Полное описание задачи: Я указал полное описание исходной задачи и детали исправления в теле pull request
  • clang-format: Мои изменения успешно проходят clang-format локально в моем форке (нет ошибок форматирования)
  • clang-tidy: Мои изменения успешно проходят clang-tidy локально в моем форке (нет предупреждений/ошибок)
  • Функциональные тесты: Все функциональные тесты успешно проходят локально на моей машине
  • Тесты производительности: Все тесты производительности успешно проходят локально на моей машине (если применимо)
  • Ветка: Я работаю в отдельной ветке для исправления (а не в master)
  • Правдивое содержание: Я подтверждаю, что все сведения, указанные в этом pull request, являются точными и достоверными

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.35294% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.59%. Comparing base (29821f1) to head (e920f8e).
⚠️ Report is 221 commits behind head on master.

Files with missing lines Patch % Lines
.../nalitov_d_dijkstras_algorithm/seq/src/ops_seq.cpp 82.35% 0 Missing and 3 partials ⚠️

❌ Your patch status has failed because the patch coverage (82.35%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1130      +/-   ##
==========================================
- Coverage   83.69%   83.59%   -0.10%     
==========================================
  Files         669      740      +71     
  Lines       34251    39256    +5005     
  Branches    14301    16379    +2078     
==========================================
+ Hits        28665    32817    +4152     
- Misses       3807     4403     +596     
- Partials     1779     2036     +257     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@allnes allnes merged commit 7a30e34 into learning-process:master May 17, 2026
34 checks passed
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.

4 participants