Skip to content

[FIXED]. Редькина Алина. ALL. Вычисление многомерных интегралов с использованием многошаговой схемы (метод Симпсона). Вариант 11.#534

Open
AlinkaRed wants to merge 4 commits intolearning-process:masterfrom
AlinkaRed:redkina_a_integral_simpson_all

Conversation

@AlinkaRed
Copy link
Copy Markdown
Contributor

Вычисление многомерных интегралов с использованием многошаговой схемы (метод Симпсона)

Задача: Вычисление многомерных интегралов с использованием многошаговой схемы (метод Симпсона)
Вариант: 11

Описание изменений:

  • Улучшила производительность.
  • Дополнила report.

Чек-лист

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

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 1, 2026

Codecov Report

❌ Patch coverage is 91.17647% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.45%. Comparing base (71b4a79) to head (d932b45).
⚠️ Report is 71 commits behind head on master.

Files with missing lines Patch % Lines
...sks/redkina_a_integral_simpson/all/src/ops_all.cpp 89.47% 1 Missing and 9 partials ⚠️
...sks/redkina_a_integral_simpson/omp/src/ops_omp.cpp 85.71% 0 Missing and 4 partials ⚠️
...sks/redkina_a_integral_simpson/stl/src/ops_stl.cpp 95.65% 0 Missing and 2 partials ⚠️
...sks/redkina_a_integral_simpson/tbb/src/ops_tbb.cpp 94.28% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #534      +/-   ##
==========================================
+ Coverage   81.36%   81.45%   +0.08%     
==========================================
  Files         407      420      +13     
  Lines       15720    16661     +941     
  Branches     6359     6761     +402     
==========================================
+ Hits        12791    13571     +780     
- Misses       2027     2147     +120     
- Partials      902      943      +41     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

#include "redkina_a_integral_simpson/common/include/common.hpp"
#include "task/include/task.hpp"

namespace redkina_a_integral_simpson {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

просьба разделить задачу и исправления

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Подскажите, пожалуйста, что вы имеете в виду? То есть сделать исправления все в отдельной ветке? А в этой просто прислать all версию?
И нужно ли для каждого исправления (omp, tbb, stl) делать отдельный pr? Или можно просто опять же с all прислать?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И, например, если мне нужно фикс сделать на omp, нужно ли остальные папки удалять?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

3 participants