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

[medical] remove collision of bandage litter #6495

Merged

Conversation

dedmen
Copy link
Contributor

@dedmen dedmen commented Aug 1, 2018

I've just observed a car crashing into a floating bandage litter and loosing it's front wheels.
All other litter items don't have collision. Just these.

@kymckay kymckay added the kind/bug-fix Release Notes: **FIXED:** label Aug 1, 2018
@kymckay kymckay added this to the Medical Rewrite milestone Aug 1, 2018
@TheMagnetar
Copy link
Member

Why? Is this not a realistic behaviour?

@JonBons
Copy link
Contributor

JonBons commented Aug 2, 2018

@dedmen I'm not sure if I'm testing this properly but it looks like littergeneric.p3d still has collision.

https://i.imgur.com/XgJdw8p.png
https://i.imgur.com/zetUxTu.jpg

Looked like the ace code actually does use ACE_MedicalLitterBase as a clutter object.

@dedmen
Copy link
Contributor Author

dedmen commented Aug 3, 2018

Oh yeah. Must've overlooked that one.

@dedmen
Copy link
Contributor Author

dedmen commented Aug 3, 2018

Also removed shadow/collision from normal items as they are too small anyway and performance.
funFact. bodyBagItem's shadowLOD was 90° rotated versus res lod.

@kymckay kymckay merged commit c1c5d6d into acemod:medical-rewrite Aug 11, 2018
BaerMitUmlaut pushed a commit that referenced this pull request Aug 5, 2019
* Remove littergeneric collision
* Remove collision/shadow from non-litter items
* Remove quik clot litter shadow
* Add empty shadowLOD to litter
@PabstMirror PabstMirror modified the milestones: Medical Rewrite, 3.13.0, 3.13.0-temp3 Dec 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
Development

Successfully merging this pull request may close these issues.

6 participants