Skip to content

Юркин Георгий.Технология ALL. Построение выпуклой оболочки – проход Грэхема. Вариант 22.#1492

Open
Georghinho wants to merge 1 commit into
learning-process:masterfrom
Georghinho:yurkin_g_var22_all_ready
Open

Юркин Георгий.Технология ALL. Построение выпуклой оболочки – проход Грэхема. Вариант 22.#1492
Georghinho wants to merge 1 commit into
learning-process:masterfrom
Georghinho:yurkin_g_var22_all_ready

Conversation

@Georghinho
Copy link
Copy Markdown
Contributor

@Georghinho Georghinho commented May 22, 2026

Описание

  • Задача: Построение выпуклой оболочки – проход Грэхема.
  • Вариант: 22
  • Технология:ALL
  • Описание вашей реализации и отчёта.
    Реализация ALL строит выпуклую оболочку методом Грэхема: входные точки сортируются по x и y (std::ranges::sort), удаляются точные дубликаты; если точек меньше двух — возвращается исходный набор. Выбирается опорная точка (минимум по y, при равенстве — по x), остальные точки сортируются по полярному углу относительно опорной (при коллинеарности — по расстоянию), все сравнения и вычисления ориентации выполняются в long double. Затем выполняется проход «со стеком»: пока ориентация трёх последних вершин <= 0 — pop_back, затем push_back текущей точки; в результате получается упорядоченный вектор вершин оболочки. Сложность 𝑂(𝑛log 𝑛), память 𝑂(𝑛)
    ; корректно обрабатываются 0–2 точки, дубликаты и коллинеарность. После вычисления детерминированного hull модуль последовательно выполняет ненарушающие результат упражнения для параллельных технологий: OpenMP (подсчёт потоков через reduction), std::thread (атомарный счётчик и join), TBB (опционально, #ifdef USE_TBB) и MPI (опционально, MPI_Barrier только при MPI_Initialized); эти блоки не модифицируют hull.

Чек-лист

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

@Georghinho Georghinho requested a review from allnes as a code owner May 22, 2026 20:39
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 82.69231% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.88%. Comparing base (8f7f008) to head (1557511).
⚠️ Report is 98 commits behind head on master.

Files with missing lines Patch % Lines
tasks/yurkin_g_graham_scan/all/src/ops_all.cpp 82.69% 2 Missing and 7 partials ⚠️

❌ Your patch status has failed because the patch coverage (82.69%) 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    #1492      +/-   ##
==========================================
- Coverage   83.50%   82.88%   -0.62%     
==========================================
  Files         815      860      +45     
  Lines       45673    49849    +4176     
  Branches    19095    20883    +1788     
==========================================
+ Hits        38138    41317    +3179     
- Misses       5079     5846     +767     
- Partials     2456     2686     +230     

☔ 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants