Обсуждение участника:IanaV: различия между версиями

Материал из Алговики
Перейти к навигации Перейти к поиску
 
(не показано 7 промежуточных версий 4 участников)
Строка 1: Строка 1:
 
п. 2.4
 
п. 2.4
  
* нет описания или ссылки на аппаратную платформу
+
* "Для вычисления производительности в неё был вписан код на MPI, реализующий подсчёт числа операций с плавающей точкой" - вписан Вами? Тогда где эта реализация? Если нет, зачем это отдельно отмечать?
* нет окончательного текста программы
+
* К графикам нет внятных пояснений.
[[Участник:Zhum|Zhum]] ([[Обсуждение участника:Zhum|обсуждение]]) 10:57, 16 ноября 2016 (MSK)
+
 
 +
Исправления:
 +
* Ссылка на окончательный текст программы была вписана еще в прошлый раз. Цитата: "Для вычисления производительности в неё был вписан код на MPI, реализующий подсчёт числа операций с плавающей точкой: <b>[https://github.com/like-a-bauss/kmeans окончательный текст программы]</b>.". Выделила жирным ссылку, чтобы было видно.
 +
* Добавлены пояснения.
 +
 
 +
Вопросы:
 +
* Эта программа последний раз изменялась несколько лет назад, судя по информации с сайта. Предложение у Вас сформулировано так, как если бы Вами в эту программу "вписан код на MPI, реализующий...". Если это так, то укажите версию с Вашими правками. Если нет, зачем писать об этой особенности программы?
 +
* Пояснения весьма скудные, практически повторяющие подписи. Требуется пояснить изменение поведения программы при изменении условий исходя из этих графиков.
 +
 
 +
Исправления:
 +
* Добавлены развёрнутые пояснения к графикам и коэффициентам масштабируемости, ссылки приведены в соответствие.
 +
 
 +
== Дополнительные замечания: ==
 +
 
 +
* добавить врезку со свойствами алгоритма
 +
* убрать звездочку как символ умножения (при сугубой необходимости, например, перед "log" можно использовать символ "\cdot")
 +
* п.1.8 функцию log лучше набирать прямым шрифтом как \log, так же как и специальне слова "distr", "update" и др., например, так: {\rm ...}
 +
* ссылка https://github.com/like-a-bauss/kmeans ведет на код с CUDA реализацией алгоритма, хотя про использование GPU нет и речи?? Уточните этот аспект и укажите опции компилятора, название очереди и, возможно, параметры GPU
 +
* ссылка [2] Lloyd S. (1957) на самом деле ведет на статью 1982 года...
 +
 
 +
Исправления:
 +
* добавлена
 +
* звездочка убрана (пункты 1.3, 1.6, 1.8, 1.9, 1.10, 2.4)
 +
* п.1.8 исправлен (аналогично пункты 1.6, 1.7, 1.10)
 +
* добавлено пояснение, что использовалась реализация на MPI (пояснения добавлены также и в описание репозитория)
 +
* в списке литературы добавлено пояснение, что статья, написанная в 1957 году, была опубликована только в 1982 году

Текущая версия на 00:39, 9 декабря 2016

п. 2.4

  • "Для вычисления производительности в неё был вписан код на MPI, реализующий подсчёт числа операций с плавающей точкой" - вписан Вами? Тогда где эта реализация? Если нет, зачем это отдельно отмечать?
  • К графикам нет внятных пояснений.

Исправления:

  • Ссылка на окончательный текст программы была вписана еще в прошлый раз. Цитата: "Для вычисления производительности в неё был вписан код на MPI, реализующий подсчёт числа операций с плавающей точкой: окончательный текст программы.". Выделила жирным ссылку, чтобы было видно.
  • Добавлены пояснения.

Вопросы:

  • Эта программа последний раз изменялась несколько лет назад, судя по информации с сайта. Предложение у Вас сформулировано так, как если бы Вами в эту программу "вписан код на MPI, реализующий...". Если это так, то укажите версию с Вашими правками. Если нет, зачем писать об этой особенности программы?
  • Пояснения весьма скудные, практически повторяющие подписи. Требуется пояснить изменение поведения программы при изменении условий исходя из этих графиков.

Исправления:

  • Добавлены развёрнутые пояснения к графикам и коэффициентам масштабируемости, ссылки приведены в соответствие.

Дополнительные замечания:

  • добавить врезку со свойствами алгоритма
  • убрать звездочку как символ умножения (при сугубой необходимости, например, перед "log" можно использовать символ "\cdot")
  • п.1.8 функцию log лучше набирать прямым шрифтом как \log, так же как и специальне слова "distr", "update" и др., например, так: {\rm ...}
  • ссылка https://github.com/like-a-bauss/kmeans ведет на код с CUDA реализацией алгоритма, хотя про использование GPU нет и речи?? Уточните этот аспект и укажите опции компилятора, название очереди и, возможно, параметры GPU
  • ссылка [2] Lloyd S. (1957) на самом деле ведет на статью 1982 года...

Исправления:

  • добавлена
  • звездочка убрана (пункты 1.3, 1.6, 1.8, 1.9, 1.10, 2.4)
  • п.1.8 исправлен (аналогично пункты 1.6, 1.7, 1.10)
  • добавлено пояснение, что использовалась реализация на MPI (пояснения добавлены также и в описание репозитория)
  • в списке литературы добавлено пояснение, что статья, написанная в 1957 году, была опубликована только в 1982 году