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

Interaction - Simplify pulling out bodies #10092

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

Conversation

johnb432
Copy link
Contributor

When merged this pull request will:

  • Title.
  • This means that the player will no longer enter the vehicle, which some might consider less realistic. Thoughts?

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

@jonpas
Copy link
Member

jonpas commented Jun 28, 2024

Reliability?

@johnb432
Copy link
Contributor Author

johnb432 commented Jun 28, 2024

Reliability?

Given that the medical interaction uses it for dead units, I imagine it's fine (testing supports that).

@Dystopian
Copy link
Contributor

some might consider less realistic

Only some?
What problem does this PR solve?

@johnb432
Copy link
Contributor Author

Only some?

Personally I don't care and as I don't want to speak for everyone, I used "some".

What problem does this PR solve?

It simplifies the function and make it more in line with the medical action's behaviour. The medical unload patient action uses moveOut for dead units, so I figured why not here too.

@Dystopian
Copy link
Contributor

I think medical magic action should be more realistic then.

@johnb432
Copy link
Contributor Author

I think medical magic action should be more realistic then.

Personally, I'm against that, as I find it cumbersome.

Unifying the behaviour should the way forward, I imagine we can agree on that. I'd like to test if using moveOut or the more realistic method is better when it comes to avoiding ragroll carnage (which was the earliest intent of the PR) before making any changes.

@johnb432 johnb432 marked this pull request as draft June 28, 2024 13:32
@Dystopian
Copy link
Contributor

Ragdoll carnage is still here when you just get in vehicle at dead body seat.

@rautamiekka
Copy link
Contributor

rautamiekka commented Jun 28, 2024

Unless you're pulling someone outta a vehicle turret or from a rear-loaded vehicle such as a troop truck or an APC, you most likely ain't going inside the vehicle anyway, so in that sense this is more realistic.

Personally don't really care if you go inside and back outside as part of the animation, but if it simplifies the code or improves performance or makes armaing less likely, then I'm all for it.

@LinkIsGrim
Copy link
Contributor

Ragdoll carnage is still here when you just get in vehicle at dead body seat.

You shouldn't be able to do that, seat should be locked when unit goes uncon/is killed. Can you repro?

@LinkIsGrim
Copy link
Contributor

Also, a good middle ground for realism would be carrying the body when pulling it out, but we've seen how that went.

Personally I'm for this, the get in/out loop to pull out a body was more annoying than anything to me, especially when the vehicle is local to a laggy machine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/optimization Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants