Skip to content

FIX: Дергунов Сергей. Технология ALL. Вычисление многомерных интегралов с использованием многошаговой схемы (метод прямоугольников). Вариант 9.#1432

Open
hhhoppe wants to merge 1 commit into
learning-process:masterfrom
hhhoppe:dergynov_s_integrals_multistep_rectangle_all_fix

Conversation

@hhhoppe
Copy link
Copy Markdown
Contributor

@hhhoppe hhhoppe commented May 21, 2026

Данные об исходной задаче

Задача: Вычисление многомерных интегралов с использованием многошаговой схемы (метод прямоугольников)
Вариант: 9
Технология: ALL

Описание исправления

Исправляется сигнатура функции ComputeALL и упрощается RunImpl для обеспечения прохождения CI.

Проблема: ALL версия вызывала все 4 реализации (SEQ, OMP, TBB, STL) на полной сетке, что приводило к превышению времени выполнения в CI (тест Product_3D падал по таймауту).

Причина: Функция ComputeALL имела нестандартную сигнатуру bool ComputeALL(..., double *out_value), а RunImpl содержал лишние проверки.

Суть исправления:

  • Сигнатура ComputeALL изменена на стандартную double ComputeALL(...)
  • RunImpl упрощён до прямого вызова ComputeALL

Проверка: Локально запущены функциональные тесты и тесты производительности для ALL версии. В личном форке запущен CI, проверено его прохождение.


  • Статус CI: Все CI-задачи (сборка, тесты, генерация отчёта) успешно проходят на моей ветке в моем форке
  • Директория и именование задачи: Я создал директорию с именем <фамилия><первая_буква_имени><короткое_название_задачи>
  • Полное описание задачи: Я предоставил полное описание задачи в теле pull request
  • clang-format: Мои изменения успешно проходят clang-format локально в моем форке (нет ошибок форматирования)
  • clang-tidy: Мои изменения успешно проходят clang-tidy локально в моем форке (нет предупреждений/ошибок)
  • Функциональные тесты: Все функциональные тесты успешно проходят локально на моей машине
  • Тесты производительности: Все тесты производительности успешно проходят локально на моей машине
  • Ветка: Я работаю в ветке, названной точно так же, как директория моей задачи (например, nesterov_a_vector_sum), а не в master
  • Правдивое содержание: Я подтверждаю, что все сведения, указанные в этом pull request, являются точными и достоверными

@hhhoppe
Copy link
Copy Markdown
Contributor Author

hhhoppe commented May 24, 2026

Вижу, что CI падает при клонировании googletest. Насколько я понимаю, это системная проблема сабмодулей, а не моих изменений.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant