Skip to content

Fix collision (knife, bullet, blood) by limiting NT collide check to only movement#689

Merged
nullsystem merged 1 commit intoNeotokyoRebuild:masterfrom
nullsystem:FixCollisionBullet
Oct 11, 2024
Merged

Fix collision (knife, bullet, blood) by limiting NT collide check to only movement#689
nullsystem merged 1 commit intoNeotokyoRebuild:masterfrom
nullsystem:FixCollisionBullet

Conversation

@nullsystem
Copy link
Copy Markdown
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 label Oct 10, 2024
@nullsystem nullsystem requested a review from a team October 10, 2024 20:00
@nullsystem nullsystem merged commit bb49e49 into NeotokyoRebuild:master Oct 11, 2024
@nullsystem nullsystem deleted the FixCollisionBullet branch May 7, 2025 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants