Skip to content

Skip redundant guide reloads during boot, keep only FML's final refresh - #41

Merged
Algent merged 1 commit into
masterfrom
algent/performances2
Jul 10, 2026
Merged

Skip redundant guide reloads during boot, keep only FML's final refresh#41
Algent merged 1 commit into
masterfrom
algent/performances2

Conversation

@Algent

@Algent Algent commented Jul 10, 2026

Copy link
Copy Markdown

Summary

Follow up on #40

Skip redundant guide reloads during boot, keep only FML's final refresh. J8 is happy, J25 probably already was stopping that somewhere (angelica ?) because impact is not measureable.

If you look for "reloadGuides currentLanguage" in fml-client-latest.log you'll see it fire three time right now, and once after this.

I couldn't see any regression from it (guide was fine in game, no missing texture or anything)

####### Daily 617
# J8 Normal First boot
[16:56:03] [Client thread/INFO] [Angelica]: The game loaded in 245.46 seconds.
# J8 Normal 2nd boot
[17:09:45] [Client thread/INFO] [Angelica]: The game loaded in 213.671 seconds.
# J8 Normal 3rd boot
[17:18:57] [Client thread/INFO] [Angelica]: The game loaded in 218.035 seconds.

# J8 NH Changed First boot
[17:04:07] [Client thread/INFO] [Angelica]: The game loaded in 219.663 seconds
# J8 NH Changed 2nd boot
[17:14:52] [Client thread/INFO] [Angelica]: The game loaded in 175.218 seconds.
# J8 NH Changed 3rd boot
[17:37:25] [Client thread/INFO] [Angelica]: The game loaded in 179.231 seconds.


# J8 NH Changed #2 First boot
[18:19:36] [Client thread/INFO] [Angelica]: The game loaded in 186.51 seconds.
# J8 NH Changed #2 2nd boot
[18:31:18] [Client thread/INFO] [Angelica]: The game loaded in 161.429 seconds.
# J8 NH Changed #2 3rd boot
[18:34:31] [Client thread/INFO] [Angelica]: The game loaded in 161.114 seconds.

1st one is before other PR, 2nd one is previous PR, last one is this PR
By first boot I mean the first one on a new instance setup (aka the worse one)

Checklist

  • I have tested this PR in DevEnv
  • I have tested this PR in Fullpack
  • This PR is in compliance with the GTNH AI Policy

@Algent Algent added the Bug Fix Fixes a bug. Please link it in the PR if an issue exists for it. label Jul 10, 2026
@Algent
Algent requested a review from ABKQPO July 10, 2026 16:41

@ABKQPO ABKQPO left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

java 8 so bad

@Algent
Algent merged commit 392e785 into master Jul 10, 2026
1 check passed
@Algent
Algent deleted the algent/performances2 branch July 10, 2026 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Fix Fixes a bug. Please link it in the PR if an issue exists for it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants