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

Luciano jacomeli patch 4.22.3 #2

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

LucianoJacomeli
Copy link

Hi Mariano, i have done some fix for your source files in 4.22.3.

I download the shooter example for 4.22.3 and use almost all classes

then i add your source code only where needs... and fix to the new version.

I change somethings on my engine source, just about public: and private: but please try. Unfortunatly only compile in shooter example downloaded from Launcher... your content not yet, i need add debugger symbols to my engine, callstack problem UE4Editor-CoreUObject Triggered Breakpoint, anyway is night in Brazil so i need sleep, tomorrow i will check it again.

i have compile with success, but i change my source file engine to fit the public: and private:
@LucianoJacomeli
Copy link
Author

Almost everything works, EQS is necessary activate in the editor preferences for work! I need help in AIController, and in EQS;

code in line 53:
"AIPerceptionComp->OnPerceptionUpdated.AddDynamic(this, &AShooterAIController::OnPerceptionUpdated);"

Also i change in EQS source master some code to let me compile, but i think is important i remove "*It" to "It"

const FVector Location = GetItemLocation(QueryInstance, It);

reactivate OnPerceptionUpdate

update TraceParams.bTraceComplex=true
@LucianoJacomeli
Copy link
Author

i will record a video, explain everything i done, but the Pointer maybe destroyed everything and crash the game! i need fix it is working but crashing. the game start became lag so i don't know if is memory leak, but a exception is called by VisualStudio

@LucianoJacomeli
Copy link
Author

Video: https://youtu.be/6AmU2oovm2E

@mtrebi mtrebi self-assigned this Jul 3, 2019
@mtrebi mtrebi added the bug label Jul 3, 2019
@mtrebi
Copy link
Owner

mtrebi commented Jul 3, 2019

OMG Good job dude! I'll be on vacation soon so I'll have time to take a look and help you fix it. Thanks :)

@LucianoJacomeli
Copy link
Author

Great news! thanks :)

@mtrebi
Copy link
Owner

mtrebi commented Aug 13, 2019

Hey, I didn't find time to do it yet sorry! I still keep it on the back of my head though.

@LucianoJacomeli
Copy link
Author

don't worry, when you have time you check. Thanks dude

@artkaste
Copy link

Bumping this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants