Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bevy update #175

Merged
merged 11 commits into from
Jul 16, 2024
Merged

Bevy update #175

merged 11 commits into from
Jul 16, 2024

Conversation

codaishin
Copy link
Owner

Update to bevy 0.14. Needed a lot of rework of existing components, especially animations. Player model has been simplified, because 2 armatures seemed to not be playing properly with bevy's new animation system.

Important: Mouse clicks (left, middle, right) cause a significant fps drop. This is likely connected to bevyengine/bevy#14303.

@codaishin codaishin force-pushed the bevy-update branch 3 times, most recently from e439a3b to 22f4ec9 Compare July 16, 2024 10:22
Reduces rounding errors of float operation with a "better" vector, so we do not need to assert approximate equality
After move to bevy 0.14, system complained about missing folder before folder contents were loaded. No error only shows after folder was loaded at least once.
@codaishin codaishin force-pushed the bevy-update branch 2 times, most recently from f901b1c to 37be082 Compare July 16, 2024 10:50
Animations didn't play any more due to the missing
newly required `AnimationTransitions` and
`AnimationGraph` components. Solved by reworking
the related systems and traits.
Previous player model could not be properly played by the new animation concept in bevy (because of the 2 model rigs?).
`MouseMotion` events not seem to work as expected. Code will be simplified and explanatory comments removed.
@codaishin codaishin merged commit db66041 into master Jul 16, 2024
3 checks passed
@codaishin codaishin deleted the bevy-update branch July 16, 2024 10:54
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