Обсуждение участника:Максим: различия между версиями
Перейти к навигации
Перейти к поиску
ASA (обсуждение | вклад) |
Lira (обсуждение | вклад) (→1.3 Замечания по содержанию от 2016.12.19: новая тема) |
||
| (не показаны 2 промежуточные версии 1 участника) | |||
| Строка 20: | Строка 20: | ||
* В разделе 1.8 должна быть оценка параллельной сложности алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 15:03, 5 декабря 2016 (MSK) | * В разделе 1.8 должна быть оценка параллельной сложности алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 15:03, 5 декабря 2016 (MSK) | ||
* В разделе 1.10 должна быть оценка вычислительной мощности алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 15:03, 5 декабря 2016 (MSK) | * В разделе 1.10 должна быть оценка вычислительной мощности алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 15:03, 5 декабря 2016 (MSK) | ||
| + | * В разделе 2.4 не приведены все параметры запуска теста - какой компилятор, с какими опциями использовался, какие версии библиотек, на каких узлах проводился запуск и т.д. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 13:27, 8 декабря 2016 (MSK) | ||
| + | * В разделе 2.4 не приведена использованная в экспериментах реализация алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 13:27, 8 декабря 2016 (MSK) | ||
| + | ** Замечание не исправлено. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 09:52, 13 декабря 2016 (MSK) | ||
| + | * Из графиков в разделе 2.4 следует сделать выводы о масштабируемости реализации. [[Участник: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) | ||
| + | |||
| + | == 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) | ||
Текущая версия на 00:17, 19 декабря 2016
Содержание
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)
