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.

Sign up for GitHub

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

Jump to bottom

Fix AI #474

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix AI #474

wants to merge 1 commit into from

Conversation

AchimTuran
Copy link
Contributor

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
Fix AI
82c54a9 
Somehow the AI held object locations got messed up.
Additionally, the playback of the locomotion animations
where to fast. It looked like Flash. :)

After this commit it should be fixed.
@AchimTuran AchimTuran mentioned this pull request Nov 24, 2023
Open
@dyanikoglu
Copy link
Owner

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

@archiekuduzu
Copy link

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.

@AchimTuran
Copy link
Contributor Author

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.

@AsifulHaque
Copy link

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

@AchimTuran
Copy link
Contributor Author

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

@AsifulHaque
Copy link

@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.

@AchimTuran
Copy link
Contributor Author

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

@AsifulHaque
Copy link

Soooo? no reply, huh? @AchimTuran

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@AchimTuran @dyanikoglu @archiekuduzu @AsifulHaque