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

fix: make clear_forces_and_impulses public #257

Merged
merged 9 commits into from
Nov 30, 2023

Conversation

ActuallyHappening
Copy link
Contributor

Objective

I am wanting to implement custom logic that runs just after ExternalForce is cleared. This helps solve issue #256.

Solution

Make the function clear_forces_and_impulses public, so that other external systems can order against it. In my case, I want to order .after(clear_forces_and_impulses).


Changelog

Changed

The function clear_forces_and_impulses was make public

src/plugins/integrator.rs Outdated Show resolved Hide resolved
src/plugins/integrator.rs Outdated Show resolved Hide resolved
src/plugins/integrator.rs Outdated Show resolved Hide resolved
ActuallyHappening and others added 4 commits November 30, 2023 10:39
to remove double line break

Co-authored-by: Joona Aalto <[email protected]>
To add trailing newline

Co-authored-by: Joona Aalto <[email protected]>
To improve documentation

Co-authored-by: Joona Aalto <[email protected]>
@ActuallyHappening
Copy link
Contributor Author

ActuallyHappening commented Nov 30, 2023

OK I believe all changes are in order
after all that, only 4 lines changed :)

src/plugins/integrator.rs Outdated Show resolved Hide resolved
@Jondolf Jondolf merged commit 7b08415 into Jondolf:main Nov 30, 2023
4 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.

2 participants