Савва Дария. Технология SEQ|MPI. Минимальное значение элементов вектора. Вариант 4#34
Conversation
Codecov Report❌ Patch coverage is ❌ 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 #34 +/- ##
===========================================
- Coverage 94.03% 15.78% -78.26%
===========================================
Files 15 189 +174
Lines 486 7256 +6770
Branches 181 2804 +2623
===========================================
+ Hits 457 1145 +688
- Misses 0 6031 +6031
- Partials 29 80 +51 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| return true; | ||
| } | ||
|
|
||
| bool SavvaDMinElemVecMPI::RunImpl() { |
There was a problem hiding this comment.
There is no data scatter from rank 0 to other ranks
| | SEQ | 2 | 9.15 | 1.00 | | | ||
| | MPI | 2 | 9.1 | 1.01 | 25,2 | | ||
| | SEQ | 8 | 24.65 | 1.00 | | | ||
| | MPI | 8 | 24.9 | 0.99 | 12,3 | | ||
| | SEQ | 4 | 12.4 | 1.00 | | | ||
| | MPI | 4 | 12.5 | 0.99 | 24,7 | |
There was a problem hiding this comment.
What does "number of processes" mean in the context of sequential execution?
| } | ||
| } | ||
|
|
||
| // случайная генерация в SetUp будет блокироваться, так как производительность зависит от сгенерированных данных |
There was a problem hiding this comment.
Этот комментарий я написала для себя как напоминание. В самом же файле main.cpp случайная генерация не используется: есть циклы, добавляющие в вектор элементы, не содержащиеся в файле data.txt, но они полностью определяются данными этого файла, то есть они одни и те же при каждом запуске.
| - Возврат итогового значения каждым процессом. | ||
| ### Ключевые функции: | ||
| - `MPI_Comm_rank`, `MPI_Comm_size` — определяют номер процесса и общее количество процессов; | ||
| - `MPI_Allreduce` — выполняет суммирование результатов всех процессов. |
There was a problem hiding this comment.
Are you sure there is a summation?
|
|
||
|
|
||
| ## Описание программной реализации параллельного алгоритма | ||
| Реализация выполнена на языке **C++17** с использованием **библиотеки MPI**. |
|
This pull request is stale for 2 weeks and is going to be closed in a week |
| int *counts = new int[size]; | ||
| int *displacements = new int[size]; |
There was a problem hiding this comment.
Please, resolve memory leaks (there are definitely no delete stmts for these allocations)
|
This pull request is stale for 2 weeks and is going to be closed in a week |
4 participants
Описание
Чек-лист
<фамилия>_<первая_буква_имени>_<короткое_название_задачи>clang-formatлокально в моем форке (нет ошибок форматирования)clang-tidyлокально в моем форке (нет предупреждений/ошибок)nesterov_a_vector_sum), а не вmaster