Обсуждение участника:Maria Zaitseva: различия между версиями
Перейти к навигации
Перейти к поиску
ASA (обсуждение | вклад) Нет описания правки |
Нет описания правки |
||
| (не показаны 2 промежуточные версии 2 участников) | |||
| Строка 1: | Строка 1: | ||
== Статья [[PAM (Partitioning Around Medoids)]] == | '''Новые замечания''' | ||
1) В верхней врезке нет свойств параллельной версии алгоритма (см страницу-образец) | |||
2) Называть множества маленькими буквами - плохая идея | |||
* Данные обозначения были сделаны для упрощения понимания. Здесь логика такая: большие буквы обозначают множества объектов, а маленькие буквы - множества индексов объектов. [[Участник:Maria Zaitseva]] 16:20, 10 декабря 2016 (MSK) | |||
3) На рис.2 показана ситуация для шага t, но буква t не участвует в подписях на рисунке | |||
== Статья [[Участник:Maria Zaitseva/PAM (Partitioning Around Medoids)]] == | |||
=== Замечания по тексту === | === Замечания по тексту === | ||
| Строка 7: | Строка 17: | ||
* В разделе 1.5 требуются указать объёмы входных и выходных данных. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 16:53, 27 октября 2016 (MSK) | * В разделе 1.5 требуются указать объёмы входных и выходных данных. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 16:53, 27 октября 2016 (MSK) | ||
** Имелся в виду раздел 1.9, конечно же. | ** Имелся в виду раздел 1.9, конечно же. | ||
* В разделе 2.4 не приведены все параметры запуска теста - какой компилятор, с какими опциями использовался, какая версия MPI, на каких узлах проводился запуск и т.д. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 15:16, 16 ноября 2016 (MSK) | |||
=== Отсутствующие части === | === Отсутствующие части === | ||
* Не указан вклад каждого из авторов статьи. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 16:53, 27 октября 2016 (MSK) | * Не указан вклад каждого из авторов статьи. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 16:53, 27 октября 2016 (MSK) | ||
Текущая версия от 13:23, 10 декабря 2016
Новые замечания
1) В верхней врезке нет свойств параллельной версии алгоритма (см страницу-образец)
2) Называть множества маленькими буквами - плохая идея
- Данные обозначения были сделаны для упрощения понимания. Здесь логика такая: большие буквы обозначают множества объектов, а маленькие буквы - множества индексов объектов. Участник:Maria Zaitseva 16:20, 10 декабря 2016 (MSK)
3) На рис.2 показана ситуация для шага t, но буква t не участвует в подписях на рисунке
Статья Участник:Maria Zaitseva/PAM (Partitioning Around Medoids)
Замечания по тексту
- Название статьи лучше дать в развёрнутом виде. Александр Сергеевич Антонов (обсуждение) 16:53, 27 октября 2016 (MSK)
- В разделе 1.4 нужно не дублировать информацию из раздела 1.3, а описать алгоритм на верхнем уровне. Александр Сергеевич Антонов (обсуждение) 16:53, 27 октября 2016 (MSK)
- В разделе 1.5 кроме псевдокода требуются текстовые пояснения. Александр Сергеевич Антонов (обсуждение) 16:53, 27 октября 2016 (MSK)
- В разделе 1.5 требуются указать объёмы входных и выходных данных. Александр Сергеевич Антонов (обсуждение) 16:53, 27 октября 2016 (MSK)
- Имелся в виду раздел 1.9, конечно же.
- В разделе 2.4 не приведены все параметры запуска теста - какой компилятор, с какими опциями использовался, какая версия MPI, на каких узлах проводился запуск и т.д. Александр Сергеевич Антонов (обсуждение) 15:16, 16 ноября 2016 (MSK)
Отсутствующие части
- Не указан вклад каждого из авторов статьи. Александр Сергеевич Антонов (обсуждение) 16:53, 27 октября 2016 (MSK)