Parallel matrix multiplication by MikePuzanov · Pull Request #1 · MikePuzanov/HomeworkSem3

MikePuzanov · 2021-09-09T19:23:12Z

No description provided.

yurii-litvinov · 2021-09-17T18:54:39Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/EmptyFileException.cs

+{
+/// <summary>
+/// then file is empty
+/// </summary>


Что-то с отступами злое

yurii-litvinov · 2021-09-17T18:55:35Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/MultiplicationException.cs

+namespace ParallelMatrixMultiplication
+{
+    /// <summary>
+    /// then we couldn't multiplication matrices


Как-то не по-английски

yurii-litvinov · 2021-09-17T18:55:50Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Program.cs

+        static void Main(string[] args)
+        {
+            var matrixTest = new int[1000, 1000];
+            Stopwatch stopWatch = new Stopwatch();


yurii-litvinov · 2021-09-17T18:55:58Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Program.cs

+            stopWatch.Start();
+            for (int i = 0; i < 10; i++)
+            {
+                var testTime =MatrixFunctions.MatrixMultiplicationParallel(matrixTest, matrixTest);


Suggested change

var testTime =MatrixFunctions.MatrixMultiplicationParallel(matrixTest, matrixTest);

var testTime = MatrixFunctions.MatrixMultiplicationParallel(matrixTest, matrixTest);

yurii-litvinov · 2021-09-17T18:57:23Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Program.cs

+            stopWatch.Stop();
+            var time = stopWatch.ElapsedMilliseconds;
+            Console.WriteLine($"Среднее время обычного умножения матриц 1000*1000: {time / 10} ms"); 
+            Console.WriteLine($"Среднее время параллельного умножения матриц 1000*1000: {timeParallel / 10} ms");


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

yurii-litvinov · 2021-09-17T19:30:35Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/FileFunctions.cs

+        {
+            if (filePath == "")
+            {
+                throw new Exception();


ArgumentException лучше. Просто Exception вообще никогда не надо кидать, его не отфильтровать потом в catch-е

yurii-litvinov · 2021-09-17T19:31:02Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/FileFunctions.cs

+                fileOut.Delete();
+            }
+            using var newFile = new FileStream(filePath, FileMode.Create);
+            var file = new StreamWriter(newFile);


Он вроде тоже IDisposable, так что using

yurii-litvinov · 2021-09-17T19:31:31Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication.Test/MatrixFunctionTest.cs

+        [Test]
+        public void TestNormalDataParallel()
+        {
+            var matrix1 = new int[1000,1000];


Пробелов после "," не хватает

yurii-litvinov · 2021-09-17T19:32:27Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication.Test/MatrixFunctionTest.cs

+            var matrix1 = new int[1000,1000];
+            var matrix2 = new int[1000,1000];
+            var result = new int[1000,1000];
+            var matrix = MatrixFunctions.MatrixMultiplicationParallel(matrix1, matrix2);


Мм, нулевые матрицы перемножаются?

yurii-litvinov · 2021-09-17T19:33:40Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Program.cs

+    {
+        static void Main(string[] args)
+        {
+            var matrixTest = new int[1000, 1000];


Непоказательно. Надо именно сгенерировать какие-нибудь матрицы с какими-нибудь адекватными числами

yurii-litvinov

Надо немного код в порядок привести

yurii-litvinov · 2021-09-29T15:26:25Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication.Test/FileFunctionsTest.cs

+        [Test]
+        public void TestNormalFilePath()
+        {
+            int[,] result = {{ 2, 1, 2}, {4, 1, 5}, {1, 5, 3}} ;


Что-то как-то пробелы то ли один лишний, то ли много не хватает. И двумерные массивы лучше на нескольких строчках писать, чтобы было в виде матрицы.

yurii-litvinov · 2021-09-29T15:29:18Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/FileFunctions.cs

+namespace ParallelMatrixMultiplication
+{
+    /// <summary>
+    /// File's function


Функция Файла? :) Какая функция и какого файла? Надо по-английски тренироваться писать (можно сначала с Google Translate или аналогичным сервисом от Яндекса, потом привыкнете и можно будет без них)

yurii-litvinov · 2021-09-29T15:34:50Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/FileFunctions.cs

+            }
+        }
+
+        private static (int, int) CountSizeMatrix(string filePath)


CountMatrixSize

ParallelMatrixMultiplication/ParallelMatrixMultiplication/MatrixFunctions.cs

+    /// </summary>
+    public static class MatrixFunctions
+    {
+        public static int[,] MatrixMultiplicationParallel(int[,] matrix1, int[,] matrix2)


yurii-litvinov · 2021-09-29T15:45:17Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Program.cs

+            Результаты на матрицах размеров 128*128.
+            Количество повторов: 100.
+            Паралельное умножение:
+            Матожидание = 13,75


Обязательно приводите размерность величин

yurii-litvinov · 2021-09-29T15:45:51Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Program.cs

+            Количество повторов: 100.
+            Паралельное умножение:
+            Матожидание = 13,75
+            Среднеквадратичное отклонение = 3,897114317029974


Не надо столько знаков после запятой. Матожидание имеет два знака, так и тут можно.

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Statistics.cs

+
+namespace ParallelMatrixMultiplication
+{
+    public class Statistics


yurii-litvinov

Тут уже число попыток близко к исчерпанию, поэтому надо, чтобы в следующий раз всё было идеально, или всё-таки минус полбалла :)

yurii-litvinov · 2021-10-09T15:34:41Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication.Test/MatrixFunctionTest.cs

+            var matrix1 = new int[3,4];
+            var matrix2 = new int[3,3];


С пробелами беда. Научитесь пользоваться https://github.com/DotNetAnalyzers/StyleCopAnalyzers, поможет :)

yurii-litvinov · 2021-10-09T15:35:34Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/FileFunctions.cs

+        {
+            if (string.IsNullOrEmpty(filePath))
+            {
+                throw new FilePathException("Error file path!");


С английским беда. "Ошибка файл путь!"

yurii-litvinov · 2021-10-09T19:11:57Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/FileFunctions.cs

+
+        private static (int, int) CountMatrixSize(string filePath)
+        {
+            CheckFilePath((filePath));


Suggested change

CheckFilePath((filePath));

CheckFilePath(filePath);

yurii-litvinov · 2021-10-09T19:16:12Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/FileFunctions.cs

+            CheckFilePath((filePath));
+            (int length, int width) size = CountMatrixSize((filePath));


Скобочки богу скобочек! https://lurkmore.to/LISP

yurii-litvinov · 2021-10-09T19:20:24Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/MatrixFunctions.cs

+                throw new MultiplicationException("Number of columns in the first matrix are not equal to the rows in the second matrix!");
+            }
+            var matrix = new int[matrix1.GetLength(0), matrix2.GetLength(1)];
+            var threads = new Thread[Environment.ProcessorCount];


Если в матрице меньше строк, чем Environment.ProcessorCount, просто зря потоки создаёте

yurii-litvinov · 2021-10-09T19:21:36Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Program.cs

+            */
+
+            var matrixFirst = FileFunctions.CreateMatrix((args[0]));
+            var matrixSecond = FileFunctions.CreateMatrix(((args[1])));


О, ещё больше скобок

yurii-litvinov · 2021-10-09T19:22:38Z

ParallelMatrixMultiplication/ParallelMatrixMultiplication/Statistics.cs

+            results[0] = (averageParallel, standardDeviationParallel);
+            results[1] = (averageNotParallel, standardDeviationNotParallel); 
+            return results;
+    }


Тут вот с отступом не повезло, а я уж было хотел зачесть задачу полностью :) Рекомендую при коммите или хотя бы когда делаете пуллреквест просматривать диффы на предмет таких косяков, они сразу в глаза бросаются

yurii-litvinov

Да, всё хорошо, зачтена

lykwer added 18 commits September 9, 2021 22:18

add FileFunctions.cs and MatrixFunctions.cs

a8c4eb9

delete and add files

9b217a4

.csproj and .sln

ba0474a

wip

8ca1603

wip

56e60b3

wip

723eb27

add two .txt

a46fb5c

appveyor.yml

3a35213

add threads and MatrixFunctionsTest.cs

350dc36

add StopWatch

8d2f7c9

code tests

033f98a

fix tests

dedd9bf

.gitignore

04d64ad

fix

37d8504

rename test

0521592

fix

70ac922

fix styleguide

715d2c1

fix

54346a5

yurii-litvinov requested changes Sep 17, 2021

View reviewed changes

fix problems

d944a26

yurii-litvinov requested changes Sep 29, 2021

View reviewed changes

lykwer added 2 commits October 1, 2021 00:39

fix code

5204153

fix code

adbe04d

yurii-litvinov requested changes Oct 9, 2021

View reviewed changes

lykwer added 3 commits October 10, 2021 17:24

fix code

5b1f8b5

fix

6bbf2d3

fix code

f09df3d

yurii-litvinov approved these changes Nov 6, 2021

View reviewed changes

	var testTime =MatrixFunctions.MatrixMultiplicationParallel(matrixTest, matrixTest);
	var testTime = MatrixFunctions.MatrixMultiplicationParallel(matrixTest, matrixTest);

		CheckFilePath((filePath));
		(int length, int width) size = CountMatrixSize((filePath));

Comments

Conversation

MikePuzanov commented Sep 9, 2021

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!

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!

yurii-litvinov left a comment

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!

This comment was marked as resolved.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment was marked as resolved.

Uh oh!

yurii-litvinov left a comment

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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

yurii-litvinov left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants