search

AlexandrZaharov / Cplusplus-Tasks

Репозиторий для домашнего задания по С++
1 stars 0 forks source link

jomework7-parallel-matrix #7

Open gitluup opened 5 years ago

gitluup commented 5 years ago

https://github.com/AlexandrZaharov/Cplusplus-Tasks/blob/73df7521fdc71932dba98528a604dcb943479a19/ParallelMatrix/ParallelMatrix/matrix.cpp#L144 Вот эту строчку для чистоты эксериментов нужно вынести в функцию main вот сюда: https://github.com/AlexandrZaharov/Cplusplus-Tasks/blob/73df7521fdc71932dba98528a604dcb943479a19/ParallelMatrix/ParallelMatrix/main.cpp#L11

https://github.com/AlexandrZaharov/Cplusplus-Tasks/blob/73df7521fdc71932dba98528a604dcb943479a19/ParallelMatrix/ParallelMatrix/matrix.cpp#L126 length вполне может быть константой

https://github.com/AlexandrZaharov/Cplusplus-Tasks/blob/73df7521fdc71932dba98528a604dcb943479a19/ParallelMatrix/ParallelMatrix/matrix.cpp#L164 Вывод чего-то на экран - это долгая и дорогая операция. А так как мы измеряем именно время перемножения матриц (в это время не должно включаться время, которое тратится на вывод чего-то в консоль), то отладочный вывод нужно убрать, когда вы производите финальные измерения (хотя в данном случае, результат наверняка почти не изменится)

https://github.com/AlexandrZaharov/Cplusplus-Tasks/blob/73df7521fdc71932dba98528a604dcb943479a19/ParallelMatrix/ParallelMatrix/matrix.cpp#L119 rhs_transposed также может быть константой

https://github.com/AlexandrZaharov/Cplusplus-Tasks/blob/73df7521fdc71932dba98528a604dcb943479a19/ParallelMatrix/ParallelMatrix/matrix.cpp#L43 Где же reserve для matrix_line?

gitluup commented 5 years ago

Хорошо