Обсуждение участника:Danyanya: различия между версиями
Перейти к навигации
Перейти к поиску
ASA (обсуждение | вклад) Нет описания правки |
Danyanya (обсуждение | вклад) |
||
| (не показано 8 промежуточных версий 4 участников) | |||
| Строка 11: | Строка 11: | ||
* В разделе 1.10 должна быть приведена оценка вычислительной сложности алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 17:44, 25 октября 2016 (MSK) | * В разделе 1.10 должна быть приведена оценка вычислительной сложности алгоритма. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 17:44, 25 октября 2016 (MSK) | ||
** Ошибся в предыдущем замечании - имелась в виду оценка вычислительной мощности. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 11:24, 1 ноября 2016 (MSK) | ** Ошибся в предыдущем замечании - имелась в виду оценка вычислительной мощности. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 11:24, 1 ноября 2016 (MSK) | ||
* Рис.4 и 5 вставлены в статью крайне неудачно. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 14:02, 21 ноября 2016 (MSK) | |||
* В разделе 2.4 не приведены все параметры запуска теста - какой компилятор, с какими опциями использовался, какие версии библиотек, на каких узлах проводился запуск и т.д. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 14:02, 21 ноября 2016 (MSK) | |||
* В разделе 2.4 не объяснены множественные особенности получившихся графиков. [[Участник:ASA|Александр Сергеевич Антонов]] ([[Обсуждение участника:ASA|обсуждение]]) 14:02, 21 ноября 2016 (MSK) | |||
=== Замечания === | |||
* убрать * в качестве умножения в математических формулах + | |||
* 1.3 хотелось бы подробнее... + | |||
* 1.6 "имеет сложность O(kn^2) и односится к квадратичным алгоритмам" - уточните, а как же "k"? + | |||
* 1.7 - хорошие графы, хотелось бы их словесного описания и, может быть, обсуждения (также чуть уменьшить поле отображения рисунков) + | |||
* 2.4 Сборка: информация "lanczos-mpi.cpp -o lanczos-mpi" совершенно лишняя + | |||
* Рис.4-5 - странные значения на шкале "Cores" + | |||
* хорошо смотрится самостоятельность реализации на С++ | |||
Текущая версия от 06:55, 7 декабря 2016
Статья Участник:Danyanya/Алгоритм Ланцоша для точной арифметики (без переортогонализации)
Отсутствующие части
- Не указан вклад каждого из авторов статьи. Александр Сергеевич Антонов (обсуждение) 17:44, 25 октября 2016 (MSK)
- Не заполнен раздел 2.4 описания. Александр Сергеевич Антонов (обсуждение) 15:10, 16 ноября 2016 (MSK)
Замечания по тексту
- В разделе 1.4 кроме выделения макроопераций требуется показать и их взаимосвязь. Александр Сергеевич Антонов (обсуждение) 17:44, 25 октября 2016 (MSK)
- Графы на рисунках в разделе 1.7 не являются графами алгоритма (граф алгоритма по определению ациклический, вершины соответствуют операторам, дуги - передаче информации между ними); рисунки должны быть пронумерованы. Александр Сергеевич Антонов (обсуждение) 17:44, 25 октября 2016 (MSK)
- Если показываете на графе (рис.1) входные и выходные данные, лучше их описать не как "Input" и "Output", а подписать конкретные элементы данных. А на рис.2 почему-то входных и выходных данных нет. Александр Сергеевич Антонов (обсуждение) 11:27, 1 ноября 2016 (MSK)
- В разделе 1.10 должна быть приведена оценка вычислительной сложности алгоритма. Александр Сергеевич Антонов (обсуждение) 17:44, 25 октября 2016 (MSK)
- Ошибся в предыдущем замечании - имелась в виду оценка вычислительной мощности. Александр Сергеевич Антонов (обсуждение) 11:24, 1 ноября 2016 (MSK)
- Рис.4 и 5 вставлены в статью крайне неудачно. Александр Сергеевич Антонов (обсуждение) 14:02, 21 ноября 2016 (MSK)
- В разделе 2.4 не приведены все параметры запуска теста - какой компилятор, с какими опциями использовался, какие версии библиотек, на каких узлах проводился запуск и т.д. Александр Сергеевич Антонов (обсуждение) 14:02, 21 ноября 2016 (MSK)
- В разделе 2.4 не объяснены множественные особенности получившихся графиков. Александр Сергеевич Антонов (обсуждение) 14:02, 21 ноября 2016 (MSK)
Замечания
- убрать * в качестве умножения в математических формулах +
- 1.3 хотелось бы подробнее... +
- 1.6 "имеет сложность O(kn^2) и односится к квадратичным алгоритмам" - уточните, а как же "k"? +
- 1.7 - хорошие графы, хотелось бы их словесного описания и, может быть, обсуждения (также чуть уменьшить поле отображения рисунков) +
- 2.4 Сборка: информация "lanczos-mpi.cpp -o lanczos-mpi" совершенно лишняя +
- Рис.4-5 - странные значения на шкале "Cores" +
- хорошо смотрится самостоятельность реализации на С++