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

Add HookEngineTick setting and EngineTick as GUI RenderMode #794

Merged
merged 3 commits into from
Feb 20, 2025

Conversation

igromanru
Copy link
Contributor

@igromanru igromanru commented Feb 17, 2025

Description
Similar to #768 where GameViewportClientTick was added as a new RenderMode for the GUI Tool, I've added EngineTick.
Additionally, I had to add HookEngineTick setting, as it wasn't a thing yet.
I put EngineTick everywhere in front of GameViewportClientTick because I think it's more important. GameViewportClient doesn't exist on servers and with Engine::Tick implementation the hook might be obsolete in the future.

How it was tested
After setting RenderMode = EngineTick, I attached the debugger to the game at the start and made sure that Engine::Tick is used instead of GameViewportClient::Tick.
Then I played around in the Live View like before and it worked well. At least as good as it usually works.
Additionally, I tested it in Dedicated Server and unlike GameViewportClientTick it works there as well! 🥳
The game in question is Abiotic Factor.
AbioticFactor-Win64-Shipping_AhZ5iE1ids

feat: Implement new RenderMode -> EngineTick
refactor: Changed gui_render_thread_GameViewportClientTick to gui_render_thread_tick, to be a generic function that can be used in all Tick function hooks
@igromanru igromanru force-pushed the rendermode_engine_tick branch from 7942e42 to a634119 Compare February 17, 2025 14:54
Copy link
Collaborator

@UE4SS UE4SS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved.
Waiting a couple days to let other people have a chance to see this.
Note to self: Clean up the two changelog commits that I created before merging.

@UE4SS UE4SS merged commit 72c00a9 into UE4SS-RE:main Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants