Обсуждение участника:IanaV: различия между версиями
Перейти к навигации
Перейти к поиску
Zhum (обсуждение | вклад) |
IanaV (обсуждение | вклад) |
||
| (не показано 7 промежуточных версий 4 участников) | |||
| Строка 1: | Строка 1: | ||
п. 2.4 | п. 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 году | ||
Текущая версия на 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 году
