Skip to content

[FIXED] Дергачёв Арсений. Технология ALL (с отчетом). Построение выпуклой оболочки - проход Грэхема. Вариант 22#567

Open
arssedot wants to merge 2 commits intolearning-process:masterfrom
arssedot:dergachev_a_graham_scan_all_fix
Open

Conversation

@arssedot
Copy link
Copy Markdown
Contributor

@arssedot arssedot commented Apr 5, 2026

Задача: Построение выпуклой оболочки – проход Грэхема

Технология: all (MPI + std::thread)

Вариант: 22

Описание

Фикс заключается в том, что я повысил производительность MPI алгоритма, немного переделав реализацию, а так же добавил отчет на ветку с all (ранее отчет был отправлен отдельным реквестом).


Checklist

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

@arssedot arssedot requested a review from allnes as a code owner April 5, 2026 11:50
@arssedot arssedot changed the title [FIX] Дергачёв Арсений. Технология ALL (с отчетом). Построение выпуклой оболочки - проход Грэхема. Вариант 22 [FIXED] Дергачёв Арсений. Технология ALL (с отчетом). Построение выпуклой оболочки - проход Грэхема. Вариант 22 Apr 5, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.66667% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.77%. Comparing base (7fd1f6d) to head (4b4c253).

Files with missing lines Patch % Lines
tasks/dergachev_a_graham_scan/all/src/ops_all.cpp 86.66% 4 Missing and 6 partials ⚠️

❌ Your patch status has failed because the patch coverage (86.66%) 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     #567      +/-   ##
==========================================
+ Coverage   81.73%   81.77%   +0.04%     
==========================================
  Files         429      429              
  Lines       17537    17598      +61     
  Branches     7165     7201      +36     
==========================================
+ Hits        14333    14390      +57     
- Misses       2251     2253       +2     
- Partials      953      955       +2     

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

@arssedot
Copy link
Copy Markdown
Contributor Author

arssedot commented Apr 6, 2026

Покрытие:

Снимок экрана 2026-04-06 160909

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