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

Movable Physics Subsystem #2880

Merged
merged 12 commits into from
May 27, 2024
Merged

Conversation

Sun-Soaked
Copy link
Member

@Sun-Soaked Sun-Soaked commented Apr 8, 2024

About The Pull Request

Ports some mojave code (originally written on Daedalus by Spyroshark) hat allows you to fling ingame objects with a simple, bouncing physics quality.
This is currently implemented on bullet casings.

Technical issues currently:

  • Makes the bouncing less dramatic

  • Physics must have a "low grav" version for tiles without gravity, since mojave didn't think gravity was real

  • The code handling the angle physics objects travel at is fucky and needs to be fixed up

Why It's Good For The Game

the hit feature from 2003, coming to a shiptest near you

Changelog

🆑 Spyroshark, Sun-Soaked
add: A movable physics subsystem, deployed using a component.
add: Bullet casings now drop using movable physics
code: ports NO_PIXEL_RANDOM_DROP from TG.
/:cl:

@keywordlabeler keywordlabeler bot added the WIP label Apr 8, 2024
@github-actions github-actions bot added DME Edit Code change Watch something violently break. labels Apr 8, 2024
@meemofcourse
Copy link
Contributor

what if we just became a mojave downstream and tested buses instead

I should've paid more attention in math class
@Sun-Soaked
Copy link
Member Author

fun fact testers: the solution to the angle problem was trigonometry. always pay attention in math class

req changes
@Sun-Soaked
Copy link
Member Author

it's slightly janky in 0g but it works now

@Sun-Soaked Sun-Soaked marked this pull request as ready for review May 18, 2024 19:55
@Sun-Soaked Sun-Soaked changed the title [WIP] Movable Physics Subsystem Movable Physics Subsystem May 18, 2024
@FalloutFalcon FalloutFalcon added Feature REALLY causes more bugs in the future that we won't know about. and removed WIP labels May 24, 2024
@FalloutFalcon FalloutFalcon added this pull request to the merge queue May 27, 2024
Merged via the queue into shiptest-ss13:master with commit abde599 May 27, 2024
14 checks passed
MysticalFaceLesS pushed a commit to CeladonSS13/Shiptest that referenced this pull request Jun 1, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->
Ports some mojave code (originally written on Daedalus by Spyroshark)
hat allows you to fling ingame objects with a simple, bouncing physics
quality.
This is currently implemented on bullet casings.

Technical issues currently:

- [x] Makes the bouncing less dramatic

- [x] Physics must have a "low grav" version for tiles without gravity,
since mojave didn't think gravity was real

- [x] The code handling the angle physics objects travel at is fucky and
needs to be fixed up

<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->
the hit feature from 2003, coming to a shiptest near you
<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

:cl: Spyroshark, Sun-Soaked
add: A movable physics subsystem, deployed using a component.
add: Bullet casings now drop using movable physics
code: ports NO_PIXEL_RANDOM_DROP from TG.
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

---------

Co-authored-by: Sun-Soaked <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break. DME Edit Feature REALLY causes more bugs in the future that we won't know about.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants