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

Материал из Алговики
Перейти к навигации Перейти к поиску
 
(не показаны 4 промежуточные версии 2 участников)
Строка 6: Строка 6:
  
 
3. Ссылки на литературу внутри текста статьи не нашел.
 
3. Ссылки на литературу внутри текста статьи не нашел.
 +
 +
 +
'''Второй этап проверки:'''
 +
Добавляйте сюда ответы на вопросы и ход исправлений.
 +
Возможно появление дополнительных замечаний.
 +
 +
1. Думаю, описание метода Пейли разумнее переместить в раздел "Математическое описание"
 +
 +
2. В математическом описании желательно хотя бы минимально кратко обосновать корректность (что достаточно проверить (p-1)/2 квадратов, и что матрица будет состоять только из элементов +1 и -1).
 +
 +
3. Исправить код. Есть как опечатки, так и ошибки. Вообще говоря, для любой похожей математической задачи необходимо писать тесты. Здесь он очевиден - проверить получившуюся матрицу по определению. В Algowiki тест вставлять не нужно, приведите его здесь.
 +
 +
4. Информационный граф неправилен. У вас, скорее, отражена макроструктура. Алгоритм довольно прост, поэтому необходимо построить граф по всем правилам, где выход операции переходит строго на вход следующей операции. Разумно будет взять конкретный пример n=4. Лучше построить граф, как с данными, так и без них. Представленный граф тоже можно будет оставить для наглядности. Его основная проблема в том, что не видно, как функция построения итоговой матрицы обращается к элементам residual, что может быть важным для каких-либо архитектур.
 +
 +
5. (22.11) с учетом ошибки и в параллельном коде, эксперимент в пункте 2.4. нужно провести заново.
 +
 +
'''Исправления по второму этапу:'''
 +
 +
На данный момент были исправлены все замечания. Скажите, пожалуйста, будут ли применяться штрафы в период правки замечаний?
 +
''Ответ:'' нет, если, как в вашем случае, происходит просто исправление, штрафов не будет. Но лучше всё же не тянуть, т.к. могут возникнуть новые комментарии.

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

Замечания к оформлению:

1. Пункт 1.7 что обозначают серые вершины на графе?

2. в пункте 1.9 указать чему соответствуют разные входные и выходные данные. Иначе непонятно будет человеку, который первый раз на сайте.

3. Ссылки на литературу внутри текста статьи не нашел.


Второй этап проверки: Добавляйте сюда ответы на вопросы и ход исправлений. Возможно появление дополнительных замечаний.

1. Думаю, описание метода Пейли разумнее переместить в раздел "Математическое описание"

2. В математическом описании желательно хотя бы минимально кратко обосновать корректность (что достаточно проверить (p-1)/2 квадратов, и что матрица будет состоять только из элементов +1 и -1).

3. Исправить код. Есть как опечатки, так и ошибки. Вообще говоря, для любой похожей математической задачи необходимо писать тесты. Здесь он очевиден - проверить получившуюся матрицу по определению. В Algowiki тест вставлять не нужно, приведите его здесь.

4. Информационный граф неправилен. У вас, скорее, отражена макроструктура. Алгоритм довольно прост, поэтому необходимо построить граф по всем правилам, где выход операции переходит строго на вход следующей операции. Разумно будет взять конкретный пример n=4. Лучше построить граф, как с данными, так и без них. Представленный граф тоже можно будет оставить для наглядности. Его основная проблема в том, что не видно, как функция построения итоговой матрицы обращается к элементам residual, что может быть важным для каких-либо архитектур.

5. (22.11) с учетом ошибки и в параллельном коде, эксперимент в пункте 2.4. нужно провести заново.

Исправления по второму этапу:

На данный момент были исправлены все замечания. Скажите, пожалуйста, будут ли применяться штрафы в период правки замечаний? Ответ: нет, если, как в вашем случае, происходит просто исправление, штрафов не будет. Но лучше всё же не тянуть, т.к. могут возникнуть новые комментарии.