Repository navigation
test: reuse per-case normalized compositor error oracle - #522
Conversation
|
No actionable comments were generated in the recent review.
 |
| Слой / файл(ы) | Сводка |
|---|---|
Вычисление ошибки и проверка масштабов test/compositor-error-bound.test.ts |
Вычисление CSS-кривой и нормализованной максимальной ошибки перенесено перед перебором масштабов. Проверки границ и расчёт ошибки в единицах масштаба остаются внутри цикла по масштабам. |
Priority:
Estimated code review effort: 2 (Simple) | ~5 minutes
Change: Other
Merge Risk: 
04f9a
-
The test optimization has no identified merge-blocking issue; merge after the stated CI and mutation qualification requirements are met.
Pre-merge checks: 
-
Check name Status Explanation Resolution Title check Заголовок точно описывает изменение, но нарушает обязательное требование русского языка: он полностью написан на английском. Заменить заголовок на краткий русский вариант, например: «test: переиспользовать нормализованный oracle ошибки compositor для каждого случая». Description check Описание содержит техническую цель, сохранённые проверки, результаты тестирования, измерения производительности и ограничения. Однако оно не соответствует обязательной структуре шаблона: отсутствуют о… Перестроить описание по шаблону репозитория. Добавить все обязательные разделы и явно указать результаты каждого применимого гейта, архитектурные проверки, риски и не-цели, документацию и связанный issue. Для неприменимых пунктов указать «н… -
Check name Status Explanation Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. краткие русские документации Добавленные комментарии в test/compositor-error-bound.test.tsнаписаны по-русски и кратко описывают новое поведение: повторное использование нормализованного oracle в пределах одного случая и вычисл…Diataxis Проверка пройдена. Pull request изменяет только test/compositor-error-bound.test.ts. Файлы документации и документационные разделы не изменены. Добавленные комментарии поясняют локальную реализацию …архитектура Проверка пройдена. Pull request изменяет только test/compositor-error-bound.test.ts. Рефакторинг переноситcompileSpringExecutionArtifactTupleUnchecked,parseCssLinearи расчёт `trueMaxErrorNorm…тесты Проверка пройдена. Diff изменяет только порядок вычислений в test/compositor-error-bound.test.ts. Для каждой пары(spring, v0)тест сохраняет 8193-точечный oracle, а затем выполняет обе проверки д…промежуточные документы (напр. планы) Проверка пройдена. В диапазоне PR изменён только test/compositor-error-bound.test.ts. Изменение переносит вычисление тестового oracle перед циклом масштабов и не добавляет планы, ревью, исследования…Full details: краткие русские документации
-
Explanation
Добавленные комментарии в
test/compositor-error-bound.test.tsнаписаны по-русски и кратко описывают новое поведение: повторное использование нормализованного oracle в пределах одного случая и вычисление кусочно-линейной кривой. Описание соответствует коду: oracle вычисляется до цикла по шкалам, а проверки границ остаются внутри этого цикла. Англоязычные комментарии и названия тестов не изменены по смыслу в этом PR.
Full details: Diataxis
-
Explanation
Проверка пройдена. Pull request изменяет только
test/compositor-error-bound.test.ts. Файлы документации и документационные разделы не изменены. Добавленные комментарии поясняют локальную реализацию теста и не являются самостоятельной документацией, требующей классификации Diataxis.
Full details: архитектура
-
Explanation
Проверка пройдена. Pull request изменяет только
test/compositor-error-bound.test.ts. Рефакторинг переноситcompileSpringExecutionArtifactTupleUnchecked,parseCssLinearи расчётtrueMaxErrorNormво внешний цикл(spring, v0), а проверкиmaxValueErrorостаются во внутреннем цикле поscale. Состояние остаётся локальным текущему тестовому случаю. Новых модулей, публичных контрактов, глобального состояния, транспортных или storage-зависимостей нет. Diff не содержит изменений production-кода и проходитgit diff --check.
Full details: тесты
-
Explanation
Проверка пройдена. Diff изменяет только порядок вычислений в
test/compositor-error-bound.test.ts. Для каждой пары(spring, v0)тест сохраняет 8193-точечный oracle, а затем выполняет обе проверки для шкал100и450. Покрыты недодемпфированный, критический, передемпфированный и медленный режимы, а также два значенияv0. Вsrc/compositor/error-bound.tsзначениеscaleне участвует в нормализованном расчёте. Оно применяется только в финальномreturn scale * maxErrorNormalized. Поэтому перенос oracle за пределы цикла шкал не ослабляет проверку. Тест способен обнаружить целевой дефект: например, саботажreturn scale * maxErrorNormalized→return -1нарушит проверку верхней границы, а удаление множителяscaleнарушит её для положительного нормализованного отклонения. Критерий функциональной доказуемости выполнен.
Full details: промежуточные документы (напр. планы)
-
Explanation
Проверка пройдена. В диапазоне PR изменён только
test/compositor-error-bound.test.ts. Изменение переносит вычисление тестового oracle перед циклом масштабов и не добавляет планы, ревью, исследования или артефакты работы агента в репозиторий продукта. Файлы документации и промежуточные документы в diff отсутствуют.
-
Full details: Description check
-
Explanation
Описание содержит техническую цель, сохранённые проверки, результаты тестирования, измерения производительности и ограничения. Однако оно не соответствует обязательной структуре шаблона: отсутствуют отдельные разделы «Пользовательский результат», «Контракт», «Архитектура», «Риски и не-цели», «Документация и выпуск», «Гейты» и «Связано с #». Статусы обязательных проверок также не оформлены в виде требуемых чекбоксов.
Resolution
Перестроить описание по шаблону репозитория. Добавить все обязательные разделы и явно указать результаты каждого применимого гейта, архитектурные проверки, риски и не-цели, документацию и связанный issue. Для неприменимых пунктов указать «не затрагивает» или привести обоснование.
-
Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
![laborant-by-labpics[bot]](https://avatars.githubusercontent.com/in/5187666?s=60&v=4){kind=small}
There was a problem hiding this comment.
Существенных проблем не нашла
Статический анализ завершён. Дополнительное ИИ-ревью выполнено.
Как проверено
разобрано файлов 1 из 1, прицельных проверок 1, прочитано файлов 17.
| Файл | Что проверено |
|---|---|
test/compositor-error-bound.test.ts |
Прочитаны все строки base и head; проверены область жизни эталона, независимость от масштаба, арифметическая эквивалентность сетки, сохранность утверждений, чистота вызываемых функций, повторное использование артефакта и выбор теста обычным и mutation-раннером. |
Изменение
Нормализованный dense-oracle вычисляется один раз внутри текущего (spring,v0), а не повторно для шкал100/450. Только test/compositor-error-bound.test.ts. Обе реальные maxValueError проверки сохранены;4пружины×2скорости×2шкалы,8193точки0..8192,parseCssLinear/solveSpring,точная арифметика времени,slack1e-10/tightness1e-6 неизменны. Нет module/pre-run cache;production/compiler cache/mutation scopes/workers/thresholds/timeouts не менялись.
Доказательства
Приёмка
Обычный requiredCI и одна полная существующая Mutation qualification на exactbranch/head до merge. Savedweeklybaseline40296355396 at1db898ab:2495mutants/3776initialtests/494staticmutants/core3752s/gate3730s/core+driver4083jobseconds. Это ещё не доказательство общего5–10min результата. Никаких повторныхbaseline63min runs или сокращения depth.
Evidence/источники
Packet D:/AgentWork/.omo/evidence/org-test-efficiency-20261023/lab-motion/weekly-numerical-oracle-20261026-st01a17463;candidate.diff SHA256cd1c97b5764fce53240f7cef14934d9c28229e12e5689b6a16f2e44ec5a8c60b (3596bytes,UTF8BOM evidence serialization),candidate sourceSHA256f39790b15d0be46880bb1329d00ae757fce16e99c0592218499e92470944cfb6. OfficialStryker staticmutants and perTest semantics:https://stryker-mutator.io/docs/mutation-testing-elements/static-mutants/ ;https://stryker-mutator.io/docs/stryker-js/configuration/ .
Rollback:revert one test-only commit. PR519browser slice remains separate;Laborant/alerting untouched.
Summary by CodeRabbit