Skip to content

Романов Артем. Технология SEQ-MPI. Интегрирование – метод прямоугольников. Вариант 19#77

Merged
allnes merged 17 commits intolearning-process:masterfrom
Quaha:romanov_a_integration_rect_method
Dec 13, 2025
Merged

Романов Артем. Технология SEQ-MPI. Интегрирование – метод прямоугольников. Вариант 19#77
allnes merged 17 commits intolearning-process:masterfrom
Quaha:romanov_a_integration_rect_method

Conversation

@Quaha
Copy link
Copy Markdown
Contributor

@Quaha Quaha commented Nov 20, 2025

Описание

  • Задача: Интегрирование – метод прямоугольников
  • Вариант: 19
  • Технология: SEQ, MPI
  • Описание:
    • Общая информация: Реализуется метод средних прямоугольников;
    • Последовательная реализация: Заданный отрезок интегрирования $[a;b]$ делится на $n$ равных частей, в цикле суммируются площади треугольников с основанием $\frac{b-a}{n}$ и высотой $f(x_i^*)$, где $x_i^*$ - центр соответствующей части отрезка интегрирования;
    • MPI версия: Считается, что функция $f$ известна всем процессам (из-за проблем её передачи). 0-ой процесс получает значения $a$, $b$ и $n$, затем их рассылает остальным процессам. После этого каждый процесс имеет одни и те же входные данные $f, a, b, n$. На основании ранга текущего процесса и общего числа процессов каждый из них вычисляет, какой подотрезок элементарных отрезков он будет обрабатывать. Каждый процесс суммирует площади на своём подотрезке, затем данные от всех процессов суммируются;
    • Отчёт: Отчёт содержит более подробную постановку задачи, описание последовательной и параллельной версий алгоритма, описание тестов и проанализированные данные о времени работы алгоритмов.

Чек-лист

  • Статус 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-commenter commented Nov 20, 2025

Codecov Report

❌ Patch coverage is 86.66667% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.22%. Comparing base (92b24d3) to head (3c2d15c).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...anov_a_integration_rect_method/mpi/src/ops_mpi.cpp 90.00% 1 Missing and 3 partials ⚠️
...anov_a_integration_rect_method/seq/src/ops_seq.cpp 78.94% 1 Missing and 3 partials ⚠️

❌ Your patch status has failed because the patch coverage (86.66%) 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      #77      +/-   ##
==========================================
- Coverage   94.03%   93.22%   -0.81%     
==========================================
  Files          15       18       +3     
  Lines         486      546      +60     
  Branches      181      199      +18     
==========================================
+ Hits          457      509      +52     
- Misses          0        2       +2     
- Partials       29       35       +6     

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

Comment thread tasks/romanov_a_integration_rect_method/mpi/src/ops_mpi.cpp
Comment thread tasks/romanov_a_integration_rect_method/seq/src/ops_seq.cpp
Comment thread tasks/romanov_a_integration_rect_method/common/include/common.hpp
Comment thread tasks/romanov_a_integration_rect_method/mpi/src/ops_mpi.cpp
Comment thread tasks/romanov_a_integration_rect_method/tests/functional/main.cpp
Comment thread tasks/romanov_a_integration_rect_method/mpi/src/ops_mpi.cpp
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do you need this file?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Данный файл действительно здесь не нужен - он остался от шаблона задачи. Удалил.


bool RomanovAIntegrationRectMethodMPI::RunImpl() {
const auto f = std::get<0>(GetInput());

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Where is the data scattering across the ranks?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Во время занятия 20.11.2025 Александр Нестеров сказал, что в задачах интегрирования объект требуемой функции std::function можно напрямую доставать из GetInput(). Рассылка остальных данных осуществляется процессом с нулевым рангом остальным ниже в коде.

Если данный комментарий был не про это - прошу описать, что не так, более подробно.

| Режим | Число процессов | Время, с | Ускорение | Эффективность |
| ----- | --------------- | -------- | --------- | ------------- |
| SEQ | 1 | 1.89 | 1.00 | N/A |
| OMP | 2 | 1.05 | 1.80 | 90.0% |
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

OMP? I suggest checking auto generated reports before publishing

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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

@Quaha Quaha requested review from allnes and aobolensk November 30, 2025 18:56
@allnes allnes merged commit 31a2505 into learning-process:master Dec 13, 2025
41 checks passed
AleksndrSakharov pushed a commit to AleksndrSakharov/ppc-2025-processes-informatics that referenced this pull request Dec 16, 2025
…иков. Вариант 19 (learning-process#77)

## Описание
- **Задача**: Интегрирование – метод прямоугольников
- **Вариант**: 19
- **Технология**: SEQ, MPI
- **Описание**:
    - **Общая информация**: Реализуется метод средних прямоугольников;
- **Последовательная реализация**: Заданный отрезок интегрирования
$[a;b]$ делится на $n$ равных частей, в цикле суммируются площади
треугольников с основанием $\frac{b-a}{n}$ и высотой $f(x_i^\*)$, где
$x_i^\*$ - центр соответствующей части отрезка интегрирования;
- **MPI версия:** Считается, что функция $f$ известна всем процессам
(из-за проблем её передачи). 0-ой процесс получает значения $a$, $b$ и
$n$, затем их рассылает остальным процессам. После этого каждый процесс
имеет одни и те же входные данные $f, a, b, n$. На основании ранга
текущего процесса и общего числа процессов каждый из них вычисляет,
какой подотрезок элементарных отрезков он будет обрабатывать. Каждый
процесс суммирует площади на своём подотрезке, затем данные от всех
процессов суммируются;
- **Отчёт**: Отчёт содержит более подробную постановку задачи, описание
последовательной и параллельной версий алгоритма, описание тестов и
проанализированные данные о времени работы алгоритмов.
---

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

5 participants