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

Материал из Алговики
Перейти к навигации Перейти к поиску

п. 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 году