Skip to content

Conversation

@DanyaLitva
Copy link

@DanyaLitva DanyaLitva commented Dec 23, 2025

  • Задача: Нахождение минимальных значений по столбцам матрицы
  • Вариант: 18
  • Технология: SEQ, MPI
  • Описание вашей реализации и отчёта.
    В параллельной версии каждому процессу столбцы делятся почти поровну между процессами(за исключением остатка). После вычисления локальных минимумов промежуточные результаты объединяются в финальный с помощью MPI_Gatherv.

Чек-лист

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

@github-actions github-actions bot added the task label Dec 23, 2025
@codecov-commenter
Copy link

codecov-commenter commented Dec 24, 2025

Codecov Report

❌ Patch coverage is 91.39785% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.52%. Comparing base (b53be17) to head (89b52ca).
⚠️ Report is 18 commits behind head on master.

Files with missing lines Patch % Lines
...vyakov_d_min_val_by_col/mpi/src/min_val_by_col.cpp 90.00% 0 Missing and 8 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   27.27%   30.52%   +3.25%     
==========================================
  Files          45       47       +2     
  Lines        1738     1831      +93     
  Branches      676      718      +42     
==========================================
+ Hits          474      559      +85     
  Misses       1232     1232              
- Partials       32       40       +8     

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

## 6. Характеристики локальной машины
- Аппаратное обеспечение/Операционная система: AMD Ryzen 5 5600, 32 GB DDR4-3200, Windows 10.
- Компилятор: MSVC, Тип сборки: Release.
- Данные: сгенерированная матрица матрица размерами 4000 на 4000 с элементами от -1e-6 до 1e-6.
Copy link
Member

Choose a reason for hiding this comment

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

Your elements are not like that in the tests

Copy link
Author

Choose a reason for hiding this comment

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

Исправлено

return std::cmp_equal(res.size(), m);
}

bool LitvyakovDMinValByColMPI::RunImpl() {
Copy link
Member

Choose a reason for hiding this comment

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

I don't see data scatter from rank 0 to other ranks

Copy link
Author

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.

@DanyaLitva DanyaLitva requested a review from aobolensk December 24, 2025 15:09
@github-actions
Copy link

This pull request is stale for 2 weeks and is going to be closed in a week

@github-actions github-actions bot added the stale label Jan 12, 2026
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