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 collision (knife, bullet, blood) by limiting NT collide check to only movement #689

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nullsystem
Copy link
Collaborator

@nullsystem nullsystem commented Oct 10, 2024

Description

Fix the collision detection which went too far. It should only be dealing with movement

Toolchain

  • Linux GCC Distro Native Arch/GCC 14

Linked Issues

@nullsystem nullsystem marked this pull request as draft October 10, 2024 17:30
@nullsystem nullsystem marked this pull request as ready for review October 10, 2024 17:50
@nullsystem nullsystem changed the title Fix collision with bullets Fix collision with bullets and knife Oct 10, 2024
@nullsystem nullsystem marked this pull request as draft October 10, 2024 18:07
@nullsystem nullsystem marked this pull request as ready for review October 10, 2024 19:20
@nullsystem nullsystem force-pushed the FixCollisionBullet branch 2 times, most recently from 85c2ee5 to 5a43859 Compare October 10, 2024 19:24
@nullsystem nullsystem changed the title Fix collision with bullets and knife Fix collision (knife, bullet, blood) by limiting to only movement Oct 10, 2024
@nullsystem nullsystem changed the title Fix collision (knife, bullet, blood) by limiting to only movement Fix collision (knife, bullet, blood) by limiting NT collide check to only movement Oct 10, 2024
@nullsystem nullsystem requested a review from a team October 10, 2024 19:30
@nullsystem nullsystem added this to the v9.0-prealpha milestone Oct 10, 2024
@nullsystem nullsystem added the bug Something isn't working label Oct 10, 2024
@nullsystem nullsystem requested a review from a team October 10, 2024 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working URGENT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants