ALS-Community icon indicating copy to clipboard operation
ALS-Community copied to clipboard
verified publisher icon dyanikoglu

Fix AI

Open AchimTuran opened this issue 2 years ago • 8 comments

Somehow the AI held object locations got messed up. Additionally, the playback of the locomotion animations where to fast. It looked like Flash. :)

Before grafik

After grafik

AchimTuran avatar Nov 24 '23 23:11 AchimTuran

Thank you, will cherry-pick this into dev branch soon.

dyanikoglu avatar Nov 28 '23 20:11 dyanikoglu

I downloaded the AI blueprint and can confirm this fixed the issue. its just a single file.

@AchimTuran do you mind telling me what changes you actually made to fix the running speed? I am curious.

archiekuduzu avatar Dec 04 '23 15:12 archiekuduzu

I downloaded the AI blueprint and can confirm this fixed the issue. its just a single file.

@AchimTuran do you mind telling me what changes you actually made to fix the running speed? I am curious.

If I rememeber correctly I just duplicated the PlayerBP and made it work againg for the AI. Then it was fixed. Maybe something got messed up during development or moving to a new UE version. Honestly I do not really know what was actually broken. As it is fixed now I did not investigate any effort further into it.

AchimTuran avatar Jan 05 '24 01:01 AchimTuran

Since @AchimTuran mentioned 'duplicating the playerBP(which I guess 'ALS_CharacterBP'), I went to see what's the difference between these two BPs, and found what was causing the problem! Inside 'ALS_AIBP', click on the mesh. On the details panel search for 'optimization', and you will find Enable Update Rate Optimizations is ticked. If this option is unticked that solves the AI weird held object issues. Tested! als_ai_fix1 als_ai_fix2

AsifulHaque avatar Feb 04 '24 22:02 AsifulHaque

@AsifulHaque Oh nice! So should I decline this PR? And you create a new one?

AchimTuran avatar Feb 20 '24 22:02 AchimTuran

@AchimTuran I don't think that's needed. I just explored a bit and found out what was the cause of the problem. But your solution should work perfectly fine too, as it basically does the same thing. Also I'm not sure these PR will ever get merged since the dev wrote in the readme that it will not get updated. This repo is sitting here for a few months without any pr merge or anything.

AsifulHaque avatar Feb 20 '24 22:02 AsifulHaque

@dyanikoglu Will you merge this pr? Or should we start a new repository to merge fixes into a master branch?

AchimTuran avatar Feb 25 '24 23:02 AchimTuran

Soooo? no reply, huh? @AchimTuran

AsifulHaque avatar Mar 01 '24 23:03 AsifulHaque

Took a bit, but finally cherry-picked this into dev branch with d765a2817f750d23286f117b277653d3fe1b244f

Will merge into main with 5.4 upgrade soon. Thank you again for the contribution!

dyanikoglu avatar Jul 09 '24 10:07 dyanikoglu