Обсуждение участника:Максим

1 Статья Участник:Максим

1.1 Отсутствующие части

• Нет заголовка с названием описываемого алгоритма. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
• В теле статьи не указаны её авторы. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
• Не заполнен раздел 1.10 описания. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
• Отсутствуют заголовки разделов 2.2-2.7 и 3. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
• Не заполнен раздел 2.4 описания. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
  • Замечание не исправлено. Александр Сергеевич Антонов (обсуждение) 15:03, 5 декабря 2016 (MSK)
• Не заполнен раздел 2.7 описания. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
  • Замечание не исправлено. Александр Сергеевич Антонов (обсуждение) 15:03, 5 декабря 2016 (MSK)
• Отсутствует список литературы. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)

1.2 Замечания по тексту

• В разделе 1.4 должно быть не перечисление макроопераций, а нужно показать структуру алгоритма на макроуровне. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
• В разделе 1.5 кроме кода нужны также текстовые пояснения. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
• В разделе 1.6 окончательно не указана последовательная сложность алгоритма. Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
• Граф, приведённый в разделе 1.7, не является информационным графом (направленный ациклический граф, вершины - операции, дуги - информационные зависимости). Александр Сергеевич Антонов (обсуждение) 13:12, 30 ноября 2016 (MSK)
  • По графу на рис.1 понять что-либо очень сложно. На рисунки три оси, но вершина расположены в узлах двумерной области. Александр Сергеевич Антонов (обсуждение) 15:03, 5 декабря 2016 (MSK)
• В разделе 1.8 должна быть оценка параллельной сложности алгоритма. Александр Сергеевич Антонов (обсуждение) 15:03, 5 декабря 2016 (MSK)
• В разделе 1.10 должна быть оценка вычислительной мощности алгоритма. Александр Сергеевич Антонов (обсуждение) 15:03, 5 декабря 2016 (MSK)
• В разделе 2.4 не приведены все параметры запуска теста - какой компилятор, с какими опциями использовался, какие версии библиотек, на каких узлах проводился запуск и т.д. Александр Сергеевич Антонов (обсуждение) 13:27, 8 декабря 2016 (MSK)
• В разделе 2.4 не приведена использованная в экспериментах реализация алгоритма. Александр Сергеевич Антонов (обсуждение) 13:27, 8 декабря 2016 (MSK)
  • Замечание не исправлено. Александр Сергеевич Антонов (обсуждение) 09:52, 13 декабря 2016 (MSK)
• Из графиков в разделе 2.4 следует сделать выводы о масштабируемости реализации. Александр Сергеевич Антонов (обсуждение) 13:27, 8 декабря 2016 (MSK)
• В разделе 2.7 нужно дать ссылки на реализации алгоритма. Александр Сергеевич Антонов (обсуждение) 13:27, 8 декабря 2016 (MSK)
• Поскольку не приведена использованная в экспериментах реализация алгоритма, непонятно даже, какие технологии при этом использовались. Опция -fopenmp задаётся для использования OpenMP, а далее написано про использование OpenMPI. Александр Сергеевич Антонов (обсуждение) 09:52, 13 декабря 2016 (MSK)

2 1.3 Замечания по содержанию от 2016.12.19

Общее замечание

Идея дать функциям разные имена (f,g, ...), а не номера (f1, f2, ...) фактически не позволяет дать описание алгоритма в общем виде.

Раздел 1.3

Описание вычислительного ядра выглядит непонятно.

Раздел 1.10

"Параллельная версия алгоритма требует в n раз меньше операций умножения и вычислений значения многомерной функции."

Это при каком числе процессоров?

Раздел 2.4

Укажите вид системы уравнений.

Полагаю, что результаты могут быть существенно улучшены и переведены в ранг содержательных при внесении двух изменений:

1) Усложнить (с точки зрения времени вычислений) вид правых частей.

2) Написать и протестировать отдельно программу для общей памяти (оставить только OpenMP).

--Lira (обсуждение) 00:17, 19 декабря 2016 (MSK)