Skip to content

Долов Вячеслав. Технология OMP. Умножение разреженных матриц. Элементы типа double. Формат хранения матрицы – строковый (CRS). Вариант 4.#502

Open
slaver07 wants to merge 17 commits intolearning-process:masterfrom
slaver07:dolov_v_crs_mat_mult_omp

Conversation

@slaver07
Copy link
Copy Markdown
Contributor

Описание

  • Задача: Умножение разреженных матриц. Элементы типа double. Формат хранения матрицы – строковый (CRS).
  • Вариант: 4
  • Технология: OMP
  • Описание
    Для решения задачи был использован алгоритм с предварительным транспонированием второй матрицы. Это позволило превратить сложный поиск элементов по столбцам в простое скалярное произведение строк, что гораздо удобнее для формата CRS. Основная работа распараллелена по строкам первой матрицы. Чтобы нагрузка на потоки распределялась равномерно, я использовал динамическое расписание dynamic.
    Внутри параллельного блока каждый поток собирает свои данные в локальные векторы, которые потом переносятся в итоговую матрицу. Это помогло избежать лишних блокировок и гонки данных. Я провел функциональные и перформанс-тесты: все проверки проходят успешно, а результаты показывают стабильное ускорение, что подтверждает корректность и эффективность реализации.

Чек-лист

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

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.38710% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 79.41%. Comparing base (97dd371) to head (662419e).

Files with missing lines Patch % Lines
tasks/dolov_v_crs_mat_mult/omp/src/ops_omp.cpp 98.38% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #502      +/-   ##
==========================================
+ Coverage   79.16%   79.41%   +0.24%     
==========================================
  Files         267      269       +2     
  Lines        9325     9442     +117     
  Branches     3963     4010      +47     
==========================================
+ Hits         7382     7498     +116     
  Misses       1374     1374              
- Partials      569      570       +1     

☔ 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.

@slaver07
Copy link
Copy Markdown
Contributor Author

Изначально рабочая директория была названа dolov_v_crs_mat_mult_seq, из-за чего падали проверки ubuntu/clang-test и ubuntu/gcc-test.

В процессе отладки я предполагал, что проблема связана с утечками памяти (по логам Valgrind), поэтому делал несколько дополнительных коммитов и много менял свой код, пытаясь это исправить.

Однако позже выяснилось, что причина была в некорректном названии рабочей директории.
После переименования в dolov_v_crs_mat_mult 23.03.2026, CI начал успешно проходить.

При этом сам алгоритм и итоговая реализация не поменялись — текущий код полностью совпадает с версией из коммита от 21.03.2026:44a4397, который был сделан до дедлайна и локально все проверки на нем проходили кроме ubuntu.

Данный PR отличается от предыдущего закрытого PR (#490) только переименованием рабочей директории, а его я открывал, потому что при похожих проблемах у некоторых студентов CI проходил, хотя падал в форке.

Прошу учесть, что задержка сдачи была связана с поиском причины падения CI, а не с доработкой алгоритма.

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