Skip to content

Conversation

@ItserX
Copy link

@ItserX ItserX commented Nov 25, 2025

Описание

  • Задача: Нахождение минимальных значений по строкам матрицы
  • Вариант: 17
  • Технология: SEQ | MPI
  • Описание
  • SEQ версия: последовательно проходимся по всем строкам матрицы, ищем в каждой строке минимальный элемент, записываем его в вектор.
  • MPI версия: распределяем строки матрицы между процессами, каждый находит минимумы в своих строках, результаты собираются на ранге 0 через MPI_Gatherv и рассылаются всем через MPI_Bcast. Пустая матрица обрабатывается корректно.

Чек-лист

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

@codecov-commenter
Copy link

codecov-commenter commented Nov 26, 2025

Codecov Report

❌ Patch coverage is 95.28302% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.25%. Comparing base (ab67334) to head (75c67c7).

Files with missing lines Patch % Lines
...ndakov_v_min_val_in_matrix_str/mpi/src/ops_mpi.cpp 94.38% 2 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
+ Coverage   94.03%   94.25%   +0.22%     
==========================================
  Files          15       17       +2     
  Lines         486      592     +106     
  Branches      181      233      +52     
==========================================
+ Hits          457      558     +101     
- Misses          0        2       +2     
- Partials       29       32       +3     

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

}

size_t cols = matrix[0].size();
// NOLINTNEXTLINE(readability-use-anyofallof)
Copy link
Member

Choose a reason for hiding this comment

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

Please, do not use NOLINTNEXTLINE(readability-use-anyofallof) in both occurrences

Copy link
Author

Choose a reason for hiding this comment

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

fixed


size_t total_rows = 0;
size_t cols = 0;
BroadcastMatrixMetadata(in_data, total_rows, cols);
Copy link
Member

Choose a reason for hiding this comment

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

Please, split the matrix and send only relevant parts 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.

fixed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants