Skip to content

fix crash on gravity inversion - #4

Merged
GoetzDeBouville merged 1 commit into
mainfrom
dev
Apr 12, 2026
Merged

fix crash on gravity inversion#4
GoetzDeBouville merged 1 commit into
mainfrom
dev

Conversation

@GoetzDeBouville

@GoetzDeBouville GoetzDeBouville commented Apr 12, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

Примечания к выпуску

  • Chores

    • Обновлена версия пакета до 2.0.1
  • Bug Fixes

    • Исправлена обработка состояния сна физических объектов
    • Улучшена обработка гравитации для тел физики: добавлена валидация масштаба гравитации с гарантией неотрицательных значений
    • Оптимизирована инициализация физических объектов при создании

@coderabbitai

coderabbitai Bot commented Apr 12, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

Обзор

Обновлена версия пакета с 2.0.0 на 2.0.1 в конфигурации Gradle. В движке физики модифицирована диспетчеризация команд, добавлено явное управление гравитационным масштабом тела и исправлена логика определения состояния сна.

Изменения

Когорта / Файл(ы) Краткое описание
Обновление версии
gradle/libs.versions.toml
Увеличение версии пакета с 2.0.0 на 2.0.1.
Логика физического движка
physicsbox/src/jvmMain/kotlin/dev/zinchenko/physicsbox/engine/PhysicsWorldEngine.kt
Изменена диспетчеризация команд на проверку типа для PhysicsCommand.ResetWorld; добавлено явное задание гравитационного масштаба при создании тела; применено ограничение гравитационного масштаба неотрицательным значением; инвертирована логика вычисления состояния сна (isAwake.not() вместо !isAwake).

Оценка сложности рецензирования

🎯 3 (Средняя) | ⏱️ ~20 минут

Стихотворение

🐰 Версия прыг-скок — со 2.0.0 на 2.0.1,
Физика танцует в новом стиле гравитации,
Тела просыпаются с верной логикой во сне,
Движок совершенствуется, как морковь ко мне!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Название точно описывает главное изменение — исправление краша при инверсии гравитации через ограничение значения gravityScale.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@GoetzDeBouville
GoetzDeBouville merged commit 4e45804 into main Apr 12, 2026
6 checks passed
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.

1 participant