Обсуждение участника:IanaV: различия между версиями
Перейти к навигации
Перейти к поиску
IanaV (обсуждение | вклад) Нет описания правки |
IanaV (обсуждение | вклад) Нет описания правки |
||
| (не показано 8 промежуточных версий 4 участников) | |||
| Строка 1: | Строка 1: | ||
п. 2.4 | |||
< | * "Для вычисления производительности в неё был вписан код на MPI, реализующий подсчёт числа операций с плавающей точкой" - вписан Вами? Тогда где эта реализация? Если нет, зачем это отдельно отмечать? | ||
* К графикам нет внятных пояснений. | |||
Исправления: | |||
* Ссылка на окончательный текст программы была вписана еще в прошлый раз. Цитата: "Для вычисления производительности в неё был вписан код на 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 году | |||
Текущая версия от 21:39, 8 декабря 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 году