View Issue Details

IDProjectCategoryView StatusLast Update
0005892The Dark ModFeature proposalpublic09.08.2022 06:21
ReporterBikerdude Assigned Tonbohr1more  
PrioritynormalSeveritynormalReproducibilityhave not tried
Status assignedResolutionopen 
PlatformPCOSWindowsOS Version10 (21H1)
Product VersionTDM 2.10 
Summary0005892: Cant mantle while carring a body
DescriptionI appreciate not being able to jump, but can we atleast mantle like we can in T1/2..?
TagsNo tags attached.

Relationships

child of 0005453 new Rethink how FM's should override cvars 

Activities

Bikerdude

Bikerdude

06.08.2022 10:21

reporter   ~0015129

If this is not possible, this tracker can be closed.
Daft Mugi

Daft Mugi

07.08.2022 04:35

reporter   ~0015141

I agree. Being able to mantle while carrying a body would feel more natural. I wonder why that was disabled.

I've attached a patch.
mantle-carry-body-v1-r10063.diff (643 bytes)   
diff --git game/Player.cpp game/Player.cpp
index 28decdb..df69a47 100644
--- game/Player.cpp
+++ game/Player.cpp
@@ -67,7 +67,7 @@ const int HEALTHPULSE_TIME = 333;
 const float MIN_BOB_SPEED = 5.0f;
 
 // shouldered body immobilizations
-const int SHOULDER_IMMOBILIZATIONS = EIM_CLIMB | EIM_ITEM_SELECT | EIM_WEAPON_SELECT | EIM_ATTACK | EIM_ITEM_USE | EIM_MANTLE | EIM_FROB_COMPLEX;
+const int SHOULDER_IMMOBILIZATIONS = EIM_CLIMB | EIM_ITEM_SELECT | EIM_WEAPON_SELECT | EIM_ATTACK | EIM_ITEM_USE | EIM_FROB_COMPLEX;
 const float SHOULDER_JUMP_HINDERANCE = 0.25f;
 
 // grayman #3485 - additional volume reduction when falling when crouched
nbohr1more

nbohr1more

08.08.2022 00:55

developer   ~0015148

I might have to convert this to a cheat cvar.
We need to verify that no current missions rely on restricting body relocation by surrounding the victim in a mantle trap.
Internal discussion started.
Bikerdude

Bikerdude

08.08.2022 07:54

reporter   ~0015151

Can said cvar be added to a config file? I would use while mapping etc.
nbohr1more

nbohr1more

08.08.2022 12:36

developer   ~0015154

There is an ongoing discussion about cvar control in general. That is issue 5453.
I had worked out a way to make mission specific cvar toggles temporary but it was a bit "Rube Goldberg" in nature so the proposed change was rejected.
Daft Mugi

Daft Mugi

08.08.2022 22:36

reporter   ~0015156

If there is a mission that has a mantle trap, that would hinder improving the movement system for all current and future missions?
nbohr1more

nbohr1more

09.08.2022 02:43

developer   ~0015157

Last edited: 09.08.2022 02:44

The general notion is that we want to try as best as possible not to break multiple missions by making them too easy or
changing the inherent tension and risk \ reward factors. That said, let's say that only one mission is impacted but the mission author
is one of our most prominent contributors. Allowing their mission to be broken by this change could cause other less prominent
mission authors to lose trust in the project.

All this said, this discussion has provoked some interesting discussions about whether "low mantle" should allow this as a realism factor
whereas mantling to high places would still not allow it. Even a change like that might cause some issues with existing missions but it may
be worth the potential upheaval to make the game richer by partially allowing this behavior.
Daft Mugi

Daft Mugi

09.08.2022 03:18

reporter   ~0015158

Thank you for explaining that. I can see how that's a tough decision.

I'd personally like to see at least a low mantle so that stepping up around knee/waist level would be possible, like stepping up onto a bed or a high step.

Would something like a DML (from Thief) be possible for old missions? That way, the engine can improve but still have a way to "fix" some older missions to be as they were originally designed. I guess something like that can already be done with an autoexec.cfg file. Maybe have a cvar that doesn't get archived, which would be some kind of mantle override to disable mantle while shouldering.

Is there a place where I can see that discussion? I think TDM is a really nice project and wonder how everyone collaborates.
Bikerdude

Bikerdude

09.08.2022 06:21

reporter   ~0015159

+1 on low mantle, you can do it in Thief 1/2.

Issue History

Date Modified Username Field Change
26.01.2022 10:16 Bikerdude New Issue
06.08.2022 10:20 Bikerdude Product Version TDM 2.09 => TDM 2.10
06.08.2022 10:21 Bikerdude Note Added: 0015129
07.08.2022 04:35 Daft Mugi Note Added: 0015141
07.08.2022 04:35 Daft Mugi File Added: mantle-carry-body-v1-r10063.diff
08.08.2022 00:55 nbohr1more Note Added: 0015148
08.08.2022 00:56 nbohr1more Assigned To => nbohr1more
08.08.2022 00:56 nbohr1more Status new => assigned
08.08.2022 07:54 Bikerdude Note Added: 0015151
08.08.2022 12:33 nbohr1more Relationship added child of 0005453
08.08.2022 12:36 nbohr1more Note Added: 0015154
08.08.2022 22:36 Daft Mugi Note Added: 0015156
09.08.2022 02:43 nbohr1more Note Added: 0015157
09.08.2022 02:43 nbohr1more Note Edited: 0015157
09.08.2022 02:44 nbohr1more Note Edited: 0015157
09.08.2022 03:18 Daft Mugi Note Added: 0015158
09.08.2022 06:21 Bikerdude Note Added: 0015159