Skip to content

Original bounding box for man#72

Open
pennane wants to merge 1 commit intomattbruv:mainfrom
pennane:pennane/man-bounding
Open

Original bounding box for man#72
pennane wants to merge 1 commit intomattbruv:mainfrom
pennane:pennane/man-bounding

Conversation

@pennane
Copy link
Contributor

@pennane pennane commented Mar 24, 2025

closes #44

in original source the bounding box was always based on the 0th image (standing man)

All the gif files have the same resolution, so I suppose the images::get_image automatically removes fully opaque pixels or something?

Nevertheless, changing to the standing image allows the man to stand in between the hangars in Katala.

Näyttökuva 2025-03-24 kello 15 31 01 Näyttökuva 2025-03-24 kello 15 36 27

@mattbruv
Copy link
Owner

Opaque pixels are not removed, but rather it looks like in the original game (and what I tried to model in this remake) is that when the collision detection happens, it first checks to see if the bounds of the images collide, and if so, depending on whether or not the entities support pixel perfect collision detection, it will then do pixel perfect collision detection to determine if they collided or not.

@mattbruv
Copy link
Owner

The only thing I wonder about is that if this is true, and it always used the standing image for collision detection, this means that even when parachuting it'd be impossible to get a kill by shooting the parachute. I'm curious if in the original it was possible to kill that way or not, I need to check into that

@pennane
Copy link
Contributor Author

pennane commented Mar 24, 2025

Yeah, originally always the standing man sprite. Based on that idsay that shooting the parachute was not possible. Also I have faint recollection of shooting down parachuters being a pain in the ass, could be wrong though.

@mattbruv
Copy link
Owner

While I agree that it looks like the original code uses just the first image for collision, they said in the discord that it was possible to kill a parachuter by shooting the parachute, so further investigation is needed to figure out how both of these things were supported

@pennane
Copy link
Contributor Author

pennane commented Mar 24, 2025

Thats interesting. Yeah some further investigation needed.

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.

Katala - unable to land between hangars

2 participants