Лаганина Елена. Задача 3. Вариант 22. Поиск кратчайших путей из одной вершины (Алгоритм Дейкстры). С CRS графов. #811

laganina-cod · 2024-12-27T18:57:28Z

Seq версия:

Алгоритм инициализирует массив расстояний, присваивая начальной вершине расстояние ноль, а всем остальным вершинам — бесконечность. Также создается массив для отслеживания посещенных вершин, изначально все вершины помечены как непосещенные. На каждом шаге алгоритм выбирает непосещенную вершину с наименьшим текущим расстоянием. Выбранная вершина отмечается как посещенная. Затем для каждой соседней вершины выбранной вершины пересчитывается расстояние от начальной вершины. Если новое вычисленное расстояние оказывается меньше, чем текущее расстояние до соседней вершины, то текущее расстояние обновляется. Данная процедура повторяется до тех пор, пока все вершины не будут посещены. По завершении алгоритма, массив расстояний содержит кратчайшие пути от начальной вершины до всех остальных вершин графа.

MPI версия:

В представленной параллельной реализации алгоритма Дейкстры для повышения вычислительной эффективности используется распределенная обработка графа между несколькими процессами. Суть подхода заключается в том, что на начальном этапе определяются все непосредственные потомки стартовой вершины, которые затем распределяются между вычислительными процессами. Каждый процесс, получив потомка, вычисляет кратчайшие расстояния от этого потомка до всех остальных вершин графа параллельно. При этом используется локальный массив расстояний, и глубина обхода графа в каждом процессе будет меньше по сравнению с последовательным алгоритмом. За счет параллельной обработки, вычислительная нагрузка на каждом процессе снижается пропорционально числу задействованных процессов. После завершения вычислений на каждом процессе, результаты собираются на управляющем процессе. На управляющем процессе также вычисляются расстояния от стартовой вершины до всех ее потомков и добавляются к полученным расстояниям. Далее происходит сбор минимальных расстояний, с использованием операции reduce, для получения окончательных значений. Полученные минимальные расстояния сравниваются с локальными данными на управляющем процессе и обновляются, таким образом, формируется финальный массив кратчайших расстояний от стартовой вершины до всех вершин графа. Основным преимуществом данного алгоритма является распараллеливание вычислений, что обеспечивает сокращение общего времени выполнения.

VladislavPoroshin

I agree with the changes

KorotinEgor

I agree with with changes

codecov-commenter · 2024-12-27T21:49:25Z

Codecov Report

Attention: Patch coverage is 92.66055% with 16 lines in your changes missing coverage. Please review.

Project coverage is 93.72%. Comparing base (041bb28) to head (97edb7c).
Report is 104 commits behind head on master.

Files with missing lines	Patch %	Lines
tasks/mpi/laganina_e_dejkstras_a/src/ops_mpi.cpp	89.47%	4 Missing and 10 partials ⚠️
...sks/mpi/laganina_e_dejkstras_a/include/ops_mpi.hpp	96.66%	0 Missing and 1 partial ⚠️
...sks/seq/laganina_e_dejkstras_a/include/ops_seq.hpp	95.45%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #811      +/-   ##
==========================================
+ Coverage   93.38%   93.72%   +0.34%     
==========================================
  Files        1090     1191     +101     
  Lines       38499    42186    +3687     
  Branches    17429    19231    +1802     
==========================================
+ Hits        35951    39540    +3589     
- Misses        904      905       +1     
- Partials     1644     1741      +97

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

allnes · 2024-12-28T21:15:19Z

tasks/mpi/laganina_e_dejkstras_a/func_tests/main.cpp

+std::vector<int> getRandomgraph(int v) {
+  std::vector<int> graph(v * v);
+  std::mt19937 engine;
+  for (int i = 0; i < v; i++) {
+    for (int j = 0; j <= i; j++) {
+      if (i == j) {
+        graph[i * v + j] = 0;
+        continue;
+      }
+      int tmp1 = engine() % 10;
+      if (tmp1 < 0) {
+        tmp1 *= -1;
+      }
+      graph[j * v + i] = graph[i * v + j] = tmp1;
+    }
+  }
+  return graph;
+}


please generate structure of graph too

The graph’s structural configuration can now be adjusted via the assignment of the requisite number of edges.

maximchhek · 2024-12-29T14:19:12Z

tasks/mpi/laganina_e_dejkstras_a/func_tests/main.cpp

@@ -0,0 +1,426 @@
+#include <gtest/gtest.h>
+
+#include <boost/mpi/communicator.hpp>


extra include

maximchhek · 2024-12-29T14:20:25Z

tasks/mpi/laganina_e_dejkstras_a/func_tests/main.cpp

+  }
+}
+
+TEST(laganina_e_dejkstras_a_mpi, Test_101_random) {


add test with 128

maximchhek · 2024-12-29T14:21:07Z

tasks/mpi/laganina_e_dejkstras_a/src/ops_mpi.cpp

@@ -0,0 +1,290 @@
+#include "mpi/laganina_e_dejkstras_a/include/ops_mpi.hpp"
+
+#include <boost/mpi/communicator.hpp>


extra includes

maximchhek · 2024-12-29T14:22:38Z

tasks/mpi/laganina_e_dejkstras_a/src/ops_mpi.cpp

+  return true;
+}
+
+void laganina_e_dejskras_a_mpi::TestMPITaskSequential::get_children_with_weights(


use static and replace function in top of file

maximchhek · 2024-12-29T14:22:46Z

tasks/mpi/laganina_e_dejkstras_a/src/ops_mpi.cpp

+  }
+}
+
+void laganina_e_dejskras_a_mpi::TestMPITaskSequential::dijkstra(int start_vertex, const std::vector<int>& row_ptr,


use static and replace function in top of file

maximchhek · 2024-12-29T14:24:13Z

tasks/seq/laganina_e_dejkstras_a/src/ops_seq.cpp

+  return true;
+}
+
+void laganina_e_dejkstras_a_Seq::laganina_e_dejkstras_a_Seq::dijkstra(int start_vertex, const std::vector<int>& row_ptr,


use static and replace function in top of file

maximchhek · 2024-12-29T14:24:39Z

tasks/seq/laganina_e_dejkstras_a/src/ops_seq.cpp

+
+#include <queue>
+#include <thread>
+#include <vector>


extra include

BrunoBucCcellati

please add a test with random graphs

BrunoBucCcellati · 2024-12-29T15:20:30Z

tasks/seq/laganina_e_dejkstras_a/func_tests/main.cpp

+  testTaskSequential.post_processing();
+  ASSERT_EQ(expectResult, trueResult);
+}
+TEST(laganina_e_dejkstras_a, Test_v_less_1) {


please add a test with random graphs

because it's seq version and correct answer for random is not know

IljarSmirnov · 2024-12-29T18:30:39Z

tasks/mpi/laganina_e_dejkstras_a/func_tests/main.cpp

+TEST(laganina_e_dejkstras_a_mpi, Test_25_random) {
+  boost::mpi::communicator world;
+  int v_ = 25;
+  int e_ = 5;


use a larger value of "e_"

maybe4

3e71ca8

laganina-cod added 6 commits December 27, 2024 22:57

maybe5

bc4bb8e

maybe6

81c56a8

maybe7

b8422e8

maybe9

7507b86

maybe10

75fadb5

maybe12

8b06641

VladislavPoroshin approved these changes Dec 27, 2024

View reviewed changes

KorotinEgor approved these changes Dec 27, 2024

View reviewed changes

allnes added the requires > 4 reviews label Dec 28, 2024

allnes reviewed Dec 28, 2024

View reviewed changes

allnes added the on review - Alexander label Dec 28, 2024

fixed

ddaf383

maximchhek reviewed Dec 29, 2024

View reviewed changes

laganina-cod added 2 commits December 29, 2024 18:13

fixed1

e994141

fixed2

0018913

maximchhek approved these changes Dec 29, 2024

View reviewed changes

fixed3

f1bed8f

BrunoBucCcellati reviewed Dec 29, 2024

View reviewed changes

laganina-cod added 4 commits December 29, 2024 19:03

fixed5

2c334ff

fixed7

fab09f4

fixed8

c539315

fixed9

976419a

IljarSmirnov reviewed Dec 29, 2024

View reviewed changes

fixed10

97edb7c

IljarSmirnov approved these changes Dec 29, 2024

View reviewed changes

BrunoBucCcellati approved these changes Dec 29, 2024

View reviewed changes

allnes added approved - Alexander and removed on review - Alexander labels Dec 29, 2024

allnes approved these changes Dec 29, 2024

View reviewed changes

allnes added wait ci and removed requires > 4 reviews labels Dec 29, 2024

aobolensk merged commit 108a937 into learning-process:master Dec 29, 2024
11 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Лаганина Елена. Задача 3. Вариант 22. Поиск кратчайших путей из одной вершины (Алгоритм Дейкстры). С CRS графов. #811

Лаганина Елена. Задача 3. Вариант 22. Поиск кратчайших путей из одной вершины (Алгоритм Дейкстры). С CRS графов. #811

laganina-cod commented Dec 27, 2024 •

edited

Loading

VladislavPoroshin left a comment

KorotinEgor left a comment

codecov-commenter commented Dec 27, 2024 •

edited

Loading

allnes Dec 28, 2024

laganina-cod Dec 29, 2024

laganina-cod Dec 29, 2024

maximchhek Dec 29, 2024

laganina-cod Dec 29, 2024

maximchhek Dec 29, 2024

laganina-cod Dec 29, 2024

maximchhek Dec 29, 2024

laganina-cod Dec 29, 2024

maximchhek Dec 29, 2024

laganina-cod Dec 29, 2024

maximchhek Dec 29, 2024

laganina-cod Dec 29, 2024

maximchhek Dec 29, 2024

laganina-cod Dec 29, 2024

maximchhek Dec 29, 2024

laganina-cod Dec 29, 2024

BrunoBucCcellati left a comment

BrunoBucCcellati Dec 29, 2024

laganina-cod Dec 29, 2024

BrunoBucCcellati Dec 29, 2024

laganina-cod Dec 29, 2024

IljarSmirnov Dec 29, 2024

laganina-cod Dec 29, 2024

		@@ -0,0 +1,426 @@
		#include <gtest/gtest.h>

		#include <boost/mpi/communicator.hpp>

		@@ -0,0 +1,290 @@
		#include "mpi/laganina_e_dejkstras_a/include/ops_mpi.hpp"

		#include <boost/mpi/communicator.hpp>

Лаганина Елена. Задача 3. Вариант 22. Поиск кратчайших путей из одной вершины (Алгоритм Дейкстры). С CRS графов. #811

Лаганина Елена. Задача 3. Вариант 22. Поиск кратчайших путей из одной вершины (Алгоритм Дейкстры). С CRS графов. #811

Conversation

laganina-cod commented Dec 27, 2024 • edited Loading

VladislavPoroshin left a comment

Choose a reason for hiding this comment

KorotinEgor left a comment

Choose a reason for hiding this comment

codecov-commenter commented Dec 27, 2024 • edited Loading

Codecov Report

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

BrunoBucCcellati left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

laganina-cod commented Dec 27, 2024 •

edited

Loading

codecov-commenter commented Dec 27, 2024 •

edited

Loading