[Request] Officially supported method of properly setting animations/ik instances.
Closed, ResolvedPublic
Actions

Description

Currently, there is no 'right' way to register an animation setup for your modded items. Many of us are doing this currently, but it has a chance to crash the server when it runs:

modded class PlayerBase
{
    override void Init()
    {            
        DayzPlayerItemBehaviorCfg     fireArmsItemBehaviour = new DayzPlayerItemBehaviorCfg;        
        fireArmsItemBehaviour.SetFirearms();
        
        GetDayZPlayerType().AddItemInHandsProfileIK("MyModdedWeapon", "dz/anims/workspaces/player/player_main/weapons/player_main_cz527.asi", fireArmsItemBehaviour, "dz/anims/anm/player/ik/weapons/cz527.anm", "dz/anims/anm/player/reloads/cz527/w_CZ527_states.anm");        
        super.Init();     
    }
}

We are 99% sure this is the cause for the crashes reported to happen on player respawn/creation on modded servers. We've tried to only set it once and not set again if already set, but it still seems to crash. Hopefully if an official way to set our animations up was in 1.08 it would eliminate a *lot* of server crashes for modded servers. I know something like this is probably coming when custom animation support arrives, but hopefully now we know this is causing crashes we can get the support to set .asi/ik early even if animation support is not ready.

Details

Severity: Feature
Resolution: Open
Reproducibility: Sometimes
Operating System: Windows 7
Category: General

Event Timeline

Ficarra1002 created this task.Apr 14 2020, 12:11 AM

CashewSan added a subscriber: CashewSan.Apr 14 2020, 12:14 AM

sanguine00 added a subscriber: sanguine00.Apr 14 2020, 12:14 AM

ContagionDayX added a subscriber: ContagionDayX.Apr 14 2020, 12:18 AM

badred96 added a subscriber: badred96.Apr 14 2020, 12:21 AM

GrosTon1 added a subscriber: GrosTon1.Apr 14 2020, 12:22 AM

Jest added a subscriber: Jest.Apr 14 2020, 12:39 AM

Unknown Object (User) added a subscriber: Unknown Object (User).Apr 14 2020, 12:49 AM

DafixCz added a subscriber: DafixCz.Apr 14 2020, 12:55 AM

Tyson added a subscriber: Tyson.Apr 14 2020, 1:10 AM

TheGL added a subscriber: TheGL.Apr 14 2020, 1:30 AM

wouter.commandeur added a subscriber: wouter.commandeur.Apr 14 2020, 10:56 AM

Salutesh added a subscriber: Salutesh.Apr 14 2020, 11:53 AM

Arkensor added a subscriber: Arkensor.Apr 14 2020, 12:58 PM

Geez changed the task status from New to Assigned.Apr 15 2020, 12:14 PM

The current function of the game for registering items IK: DayZPlayerTypeRegisterItems(DayZPlayerType Type) is a global function.

And we can't mod this function, that's why modders can't easily add new IK animation. If this function would be inside a class, we could easily mod it.
So you just need to create a new class and move this function into it.
Don't forget we can't mod static function, so it's should not be a static function.

Thanks for the assignment.

As said in https://feedback.bistudio.com/T149787, went from over 24 crashes a day to one and after 18,5 hours of uptime, this is definitely what causes the crashes on player spawns events.

Fabian.- added a subscriber: Fabian.-.Apr 16 2020, 7:11 AM

Added a temporary feature into Community Framework, in 4_World use this for now. This should hopefully resolve issues until a proper and crash free solution is implemented.

Methods (don't forget to call super): https://github.com/Jacob-Mango/DayZ-Community-Framework/blob/steam/JM/CF/Scripts/4_World/Classes/JMAnimRegister.c#L56-L110

How to use:

modded class JMAnimRegister
{
  override void OnRegisterFireArms( DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior )
  {
    super.OnRegisterFireArms( pType, pBehavior );
    pType..AddItemInHandsProfileIK("MyModdedWeapon", "dz/anims/workspaces/player/player_main/weapons/player_main_cz527.asi", pBehavior, "dz/anims/anm/player/ik/weapons/cz527.anm", "dz/anims/anm/player/reloads/cz527/w_CZ527_states.anm");    
  }
};

This hasn't been tested to see if it does fix the crashes but we have used a similar solution in the Expansion mod with no problems yet.

lagtek13012 added a subscriber: lagtek13012.Apr 28 2020, 2:45 AM

Xairo added a subscriber: Xairo.May 4 2020, 10:15 PM

Hello Ficarra1002.
In 1.08 it is going to be possible to do this with the following class:

class ModItemRegisterCallbacksclass ModItemRegisterCallbacks
{

void RegisterEmptyHanded(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterOneHanded(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterTwoHanded(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterRestrained(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterSurrender(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterHeavy(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterPistol(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterFireArms(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterArcheryItem(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterLauncher(DayZPlayerType pType, DayzPlayerItemBehaviorCfg pBehavior);
void RegisterCustom(DayZPlayerType pType);
void CustomBoneRemapping(DayZPlayerType pType);

}

Regards,
Geez

This is amazing news, thank's to everyone that probably worked hard and fast to get this in this fast !

Regards.

PR9INICHEK added a subscriber: PR9INICHEK.May 5 2020, 1:28 PM

mordecai__1 added a subscriber: mordecai__1.Aug 19 2021, 8:22 AM

[Request] Officially supported method of properly setting animations/ik instances. Closed, ResolvedPublicActions

Description

Details

Event Timeline

[Request] Officially supported method of properly setting animations/ik instances.
Closed, ResolvedPublic
Actions