Гусева Алена. Технология SEQ-MPI. Сумма элементов матрицы по столбцам. Вариант 12 #11

gsvval · 2025-10-29T19:07:37Z

Описание

Задача: Сумма элементов матрицы по столбцам
Вариант: 12
Технология: SEQ, MPI
Описание вашей реализации и отчёта.
SEQ: Создается вектор сумм столбцом входной матрицы. Итерируясь по матрице, каждый очередной ее элемент добавляется к соответствующей сумме.
MPI: Элементы входной матрицы поровну распределяются по процессам. Если число элементов не кратно числу процессов, нераспределенные n элементов дораспределяются по первым n процессам. Рассчитываются массивы количеств и смещений для MPI_Scatterv() и локальный сдвиг shift для корректного суммирования текущих элементов с соответствующими элементами локального результирующего вектора (для случаев, когда границы распределенных диапазонов не соответствуют границам строк матрицы). Когда локальные результаты вычислены, они собираются на корневом процессе функцией MPI_Reduce.

Чек-лист

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

gsvval · 2025-10-29T20:06:29Z

Отправила ПР, т.к. локально проверить не могу, по какой-то причине нет доступных раннеров:
https://github.com/gsvval/ppc-2025-processes-informatics/actions/runs/18918876144

@allnes
@aobolensk

aobolensk · 2025-10-30T14:21:02Z

Отправила ПР, т.к. локально проверить не могу, по какой-то причине нет доступных раннеров: https://github.com/gsvval/ppc-2025-processes-informatics/actions/runs/18918876144 @allnes @aobolensk

That could be related to the major outage of Microsoft Azure yesterday. Now that should be OK

codecov-commenter · 2025-10-31T10:10:07Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.14%. Comparing base (858535d) to head (f480a05).

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   94.58%   95.14%   +0.56%     
==========================================
  Files          15       17       +2     
  Lines         480      536      +56     
  Branches      177      201      +24     
==========================================
+ Hits          454      510      +56     
  Partials       26       26

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

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

tasks/guseva_a_matrix_sums/mpi/src/ops_mpi.cpp

Update to current upstream after git reset

gsvval · 2025-11-06T16:03:25Z

Подскажите, пожалуйста, как лишние Labels убрать? Делала git reset и push --force после pull'a хотфиксов. В списке измененных файлов только моя директория и мои файлы
@aobolensk

aobolensk · 2025-11-06T17:12:21Z

Подскажите, пожалуйста, как лишние Labels убрать? Делала git reset и push --force после pull'a хотфиксов. В списке измененных файлов только моя директория и мои файлы
@aobolensk

No worries, fixed manually

aobolensk · 2025-11-06T20:28:19Z

tasks/guseva_a_matrix_sums/report.md

Report is not attached

Added with the last commit

tasks/guseva_a_matrix_sums/tests/functional/main.cpp

aobolensk · 2025-11-06T21:50:45Z

tasks/guseva_a_matrix_sums/mpi/src/ops_mpi.cpp

+  if (rank == 0) {
+    GetOutput() = global_sums;
+  } else {
+    GetOutput() = {-1};


Output is not actually required to be initialized for other ranks

Actualize repo's state

…termidiate vector for global sums

DianaRych · 2025-11-09T19:53:52Z

tasks/guseva_a_matrix_sums/mpi/src/ops_mpi.cpp

+    local_sums[(i + shift) % columns] += local_buff[i];
+  }
+
+  std::vector<double> global_sums(columns, 0.0);


Можно использовать MPI_Reduce сразу для GetOutput()

ChastovSlava · 2025-11-09T20:02:19Z

tasks/guseva_a_matrix_sums/seq/src/ops_seq.cpp

+  auto &matrix = std::get<2>(GetInput());
+  auto &columns = std::get<1>(GetInput());
+  auto &rows = std::get<0>(GetInput());


Можно распаковать кортеж в три переменные одной строкой

gsvval requested review from allnes and aobolensk as code owners October 29, 2025 19:07

github-actions bot added the task label Oct 29, 2025

gsvval marked this pull request as draft October 30, 2025 15:45

aobolensk marked this pull request as ready for review October 31, 2025 18:31

aobolensk marked this pull request as draft October 31, 2025 18:31

chatgpt-codex-connector bot reviewed Oct 31, 2025

View reviewed changes

tasks/guseva_a_matrix_sums/mpi/src/ops_mpi.cpp Outdated Show resolved Hide resolved

gsvval marked this pull request as ready for review November 6, 2025 15:06

gsvval force-pushed the guseva_a_matrix_sums branch from 5b308b8 to b57dede Compare November 6, 2025 15:09

github-actions bot added python Pull requests that update python code docs ci infrastructure scoreboard modules:task modules:util labels Nov 6, 2025

[init]

0c5de27

gsvval force-pushed the guseva_a_matrix_sums branch from b57dede to 0c5de27 Compare November 6, 2025 15:11

Merge pull request #4 from learning-process/master

f480a05

Update to current upstream after git reset

aobolensk removed python Pull requests that update python code docs ci infrastructure scoreboard modules:task modules:util labels Nov 6, 2025

aobolensk reviewed Nov 6, 2025

View reviewed changes

gsvval added 2 commits November 8, 2025 04:42

Merge pull request #6 from learning-process/master

8cbeaaa

Actualize repo's state

[fix] Fix all from @aobolensk review

ac08e3f

gsvval requested a review from aobolensk November 8, 2025 15:07

[refactor] Fix typo

dc589ad

aobolensk approved these changes Nov 8, 2025

View reviewed changes

[fix] Use structured bindings for tuple unpacking, Remove redunant in…

0055632

…termidiate vector for global sums

DianaRych suggested changes Nov 9, 2025

View reviewed changes

ChastovSlava suggested changes Nov 9, 2025

View reviewed changes

gsvval requested review from ChastovSlava and DianaRych November 10, 2025 00:48

ChastovSlava approved these changes Nov 10, 2025

View reviewed changes

DianaRych approved these changes Nov 11, 2025

View reviewed changes

Гусева Алена. Технология SEQ-MPI. Сумма элементов матрицы по столбцам. Вариант 12 #11

Are you sure you want to change the base?

Гусева Алена. Технология SEQ-MPI. Сумма элементов матрицы по столбцам. Вариант 12 #11

Uh oh!

Conversation

gsvval commented Oct 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Описание

Чек-лист

Uh oh!

gsvval commented Oct 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

aobolensk commented Oct 30, 2025

Uh oh!

codecov-commenter commented Oct 31, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

gsvval commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

aobolensk commented Nov 6, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

gsvval commented Oct 29, 2025 •

edited

Loading

gsvval commented Oct 29, 2025 •

edited

Loading

codecov-commenter commented Oct 31, 2025 •

edited

Loading

gsvval commented Nov 6, 2025 •

edited

Loading

aobolensk commented Nov 6, 2025 •

edited

Loading