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

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

--Vvf63 (обсуждение) 00:06, 1 ноября 2016 (MSK)

1 Статья Участник:Pandvik/Ортогонализация Грама - Шмидта

1.1 Отсутствующие части

1.2 Замечания по тексту

  • В разделе 1.4 требуется привести описание алгоритма на верхнем уровне (на основе макроопераций). Александр Сергеевич Антонов (обсуждение) 17:27, 31 октября 2016 (MSK)
  • Структура графов на рис.1-3 непонятна. Если двойными кругами показаны элементы данных, то лучше их разделить на входные и выходные. Не всё понятно и с дугами, например, на рис.3 оранжевым овалом обозначаются скалярные произведения двух векторов, но в некоторые из овалов входит только одна дуга, а откуда берётся второй аргумент? Александр Сергеевич Антонов (обсуждение) 17:27, 31 октября 2016 (MSK)

1.3 DONE

  • Обновлены картинки (добавлены/исправлены стрелки). Добавлены потерявшиеся стрелки; Изменено направление стрелок для большей наглядности; Входные и выходные данные окрашены в разные цвета, а назначения раскрашенные указано в тексте раздела. --Pandvik (обсуждение) 23:40, 31 октября 2016 (MSK)
    • Граф нужен не сам по себе, а чтобы наглядно показать структуру алгоритма. Граф на рис.3, по-моему, максимально запутанный, понято по нему что-либо очень сложно. Александр Сергеевич Антонов (обсуждение) 18:11, 1 ноября 2016 (MSK)
      • Мне кажется, что для описания основной структуры алгоритма достаточно Рис.1. Этот рисунок не загромождён лишними данными и позволяет оценить возможности параллелизма для данного алгоритма. Рисунок 3 является попыткой более подробно составить граф зависимости, и приводит к слишком сильному загромождению картинки стрелками зависимостей. Я постарался переделать этот рисунок. Исправил ошибки, переделал и раскрасил стрелки. Возможно, он стал выглядеть немного получше. --Pandvik (обсуждение) 09:47, 3 ноября 2016 (MSK)
  • Внесено уточнение в макроструктуру алгоритма (1.4). Было добавлено пояснение к преобладающим операциям алгоритма. --Vvf63 (обсуждение) 00:06, 1 ноября 2016 (MSK)