Обсуждение участника:Максим: различия между версиями
Перейти к навигации
Перейти к поиску
ASA (обсуждение | вклад) Нет описания правки |
Lira (обсуждение | вклад) →1.3 Замечания по содержанию от 2016.12.19: новая тема |
||
| Строка 26: | Строка 26: | ||
* В разделе 2.7 нужно дать ссылки на реализации алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 13:27, 8 декабря 2016 (MSK) | * В разделе 2.7 нужно дать ссылки на реализации алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 13:27, 8 декабря 2016 (MSK) | ||
* Поскольку не приведена использованная в экспериментах реализация алгоритма, непонятно даже, какие технологии при этом использовались. Опция -fopenmp задаётся для использования OpenMP, а далее написано про использование OpenMPI. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 09:52, 13 декабря 2016 (MSK) | * Поскольку не приведена использованная в экспериментах реализация алгоритма, непонятно даже, какие технологии при этом использовались. Опция -fopenmp задаётся для использования OpenMP, а далее написано про использование OpenMPI. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 09:52, 13 декабря 2016 (MSK) | ||
== 1.3 Замечания по содержанию от 2016.12.19 == | |||
Общее замечание | |||
Идея дать функциям разные имена (f,g, ...), а не номера (f1, f2, ...) фактически не позволяет дать описание алгоритма в общем виде. | |||
Раздел 1.3 | |||
Описание вычислительного ядра выглядит непонятно. | |||
Раздел 1.10 | |||
"Параллельная версия алгоритма требует в n раз меньше операций умножения и вычислений значения многомерной функции." | |||
Это при каком числе процессоров? | |||
Раздел 2.4 | |||
Укажите вид системы уравнений. | |||
Полагаю, что результаты могут быть существенно улучшены и переведены в ранг содержательных при внесении двух изменений: | |||
1) Усложнить (с точки зрения времени вычислений) вид правых частей. | |||
2) Написать и протестировать отдельно программу для общей памяти (оставить только OpenMP). | |||
--[[Участник:Lira|Lira]] ([[Обсуждение участника:Lira|обсуждение]]) 00:17, 19 декабря 2016 (MSK) | |||
Текущая версия от 21:17, 18 декабря 2016
Статья Участник:Максим
Отсутствующие части
- Нет заголовка с названием описываемого алгоритма. Александр Сергеевич Антонов (обсуждение) 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.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)
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)