Skip to content

[FIXED] Перепелкин Ярослав. Технология OMP. Построение выпуклой оболочки – проход Грэхема. Вариант 22#671

Open
TepidmishA wants to merge 6 commits intolearning-process:masterfrom
TepidmishA:perepelkin_i_convex_hull_graham_scan_omp_fix
Open

[FIXED] Перепелкин Ярослав. Технология OMP. Построение выпуклой оболочки – проход Грэхема. Вариант 22#671
TepidmishA wants to merge 6 commits intolearning-process:masterfrom
TepidmishA:perepelkin_i_convex_hull_graham_scan_omp_fix

Conversation

@TepidmishA
Copy link
Copy Markdown
Contributor

Описание

  • Задача: Построение выпуклой оболочки – проход Грэхема
  • Вариант: 22
  • Технология: OMP

Изменения

  • Последовательное построение оболочки вынесено в метод HullConstruction
  • Убрано использование std::ranges::sort для входных данных меньше 10k элементов, чтобы в функциональных тестах использовалась параллельная сортировка OMP
  • Критическая секция с редукцией в FindPivotParallel заменена на последовательное объединение результатов
  • Улучшено распределение данных между потоками в ParallelSort

Чек-лист

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

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.36364% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.31%. Comparing base (f7f3d46) to head (daefded).

Files with missing lines Patch % Lines
...lkin_i_convex_hull_graham_scan/omp/src/ops_omp.cpp 86.36% 1 Missing and 2 partials ⚠️

❌ Your patch status has failed because the patch coverage (86.36%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #671      +/-   ##
==========================================
+ Coverage   82.27%   82.31%   +0.04%     
==========================================
  Files         497      497              
  Lines       21191    21201      +10     
  Branches     8691     8691              
==========================================
+ Hits        17434    17452      +18     
+ Misses       2614     2605       -9     
- Partials     1143     1144       +1     

☔ 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.

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.

2 participants