Skip to content

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

Merged
allnes merged 18 commits into
learning-process:masterfrom
Georghinho:yurkin_all_and_reports
Jun 4, 2026
Merged

Юркин Георгий.Технология ALL + Отчеты. Построение выпуклой оболочки – проход Грэхема. Вариант 22.#1548
allnes merged 18 commits into
learning-process:masterfrom
Georghinho:yurkin_all_and_reports

Conversation

@Georghinho

@Georghinho Georghinho commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Описание

  • Задача: Построение выпуклой оболочки – проход Грэхема.
  • Вариант: 22
  • Технология: ALL+Reports
  • Описание вашей реализации и отчёта.
    Реализация ALL строит выпуклую оболочку методом Грэхема: входные точки сортируются по x и y (std::ranges::sort), удаляются точные дубликаты; если точек меньше двух — возвращается исходный набор. Выбирается опорная точка (минимум по y, при равенстве — по x), остальные точки сортируются по полярному углу относительно опорной (при коллинеарности — по расстоянию), все сравнения и вычисления ориентации выполняются в long double. Затем выполняется проход «со стеком»: пока ориентация трёх последних вершин <= 0 — pop_back, затем push_back текущей точки; в результате получается упорядоченный вектор вершин оболочки. Сложность 𝑂(nlog n), память 𝑂(n)
    ; корректно обрабатываются 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, являются точными и
    достоверными

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 77.58621% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.05%. Comparing base (8f7f008) to head (828f797).
⚠️ Report is 346 commits behind head on master.

Files with missing lines Patch % Lines
tasks/yurkin_g_graham_scan/all/src/ops_all.cpp 77.58% 4 Missing and 9 partials ⚠️

❌ Your patch status has failed because the patch coverage (77.58%) 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    #1548      +/-   ##
==========================================
- Coverage   83.50%   83.05%   -0.45%     
==========================================
  Files         815        9     -806     
  Lines       45673      419   -45254     
  Branches    19095      252   -18843     
==========================================
- Hits        38138      348   -37790     
+ Misses       5079       20    -5059     
+ Partials     2456       51    -2405     

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

@allnes allnes merged commit 588ff17 into learning-process:master Jun 4, 2026
37 checks passed
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