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

[Bug Report]: Client hang when picking up an item for the first time #323

Open
James103 opened this issue Dec 10, 2024 · 9 comments
Open
Assignees
Labels
bug Something isn't working

Comments

@James103
Copy link
Contributor

James103 commented Dec 10, 2024

Modpack Version

1.9.0

Describe your issue.

When picking up an item for the first time after restarting a Craftoria client, the client hangs for around 5 seconds, all of which is spent rebuilding the item tabs/groups normally used in Creative Mode. This occurs even in Survival mode because the Loot Journal mod triggers the rebuild, not opening the Creative inventory.

Approximately half of that time is spent in Log Begone filtering out log messages produced during the creative inventory tab rebuild. See the attached spark profile for more information.

A similar hang occurs when using Inventory Profiles Next's middle-click functionality to sort a container or when going into Creative Mode and opening the Creative Mode inventory; if either of those occur first, picking up an item does not cause an additional hang.

Spark profiles

Picking up an item: 10fmfVR6GP.zip

Sorting a container with Inventory Profiles Next: X1UsC4P4k0.zip

Crash Report

No response

Latest Log

No response

Have you modified the modpack?

Yes

User Modifications

Remove: Champions Unofficial (to reduce difficulty)
Add: No Chat Reports 1.21.1-v1.29.1
Add: Xaero's Minimap 24.6.1 and Xaero's World Map 1.39.0
Add: Tom's Simple Storage Mod 2.0.10

Did the issue happen in singleplayer or on a server?

Singleplayer

Discord Username

No response

@James103 James103 added the bug Something isn't working label Dec 10, 2024
@Gronglegrowth
Copy link

Oh is that why?

@SubordinalBlue
Copy link
Contributor

SubordinalBlue commented Dec 10, 2024

Great find/diagnosis... but shouldn't this be a bug report with Loot Journal?

@James103
Copy link
Contributor Author

Great find/diagnosis... but shouldn't this be a bug report with Loot Journal?

Loot Journal has no public-facing issue tracker outside the mod developer's Discord server.

@SubordinalBlue
Copy link
Contributor

SubordinalBlue commented Dec 10, 2024

Well... did you report the bug to them? (GH, Discord, whatever... surely they take bug reporting in some form?)

Unless I'm misunderstanding (always a good chance of that, tbf), the only action available to the pack on this is to remove Loot Journal... which doesn't seem like the optimal outcome.

@MechanicalDragon0687
Copy link

Does this issue occur if you sort a chest with inventory profiles next in creative mode sort order (default) before picking up an item?

@James103
Copy link
Contributor Author

Does this issue occur if you sort a chest with inventory profiles next in creative mode sort order (default) before picking up an item?

Yes, the same hang occurs and the attached profiler results show the creative tab content rebuild being triggered by Inventory Profiles Next's item sorting function.
https://spark.lucko.me/X1UsC4P4k0
X1UsC4P4k0.zip

@MechanicalDragon0687
Copy link

Does this issue occur if you sort a chest with inventory profiles next in creative mode sort order (default) before picking up an item?

Yes, the same hang occurs and the attached profiler results show the creative tab content rebuild being triggered by Inventory Profiles Next's item sorting function. https://spark.lucko.me/X1UsC4P4k0 X1UsC4P4k0.zip

but does the item pickup lag happen after IPN has forced the creative tab groups to generate by using it to sort?
open container
sort
check if first pickup still lags

@James103
Copy link
Contributor Author

check if first pickup still lags

I do not get any additional lag upon picking up an item for the first time, if the creative tabs were already generated by usage of Inventory Profiles Next's sort function or opening the inventory in Creative Mode.

@MechanicalDragon0687
Copy link

Someone might be able to make a mod that generates those tabs at join, then, to put the lag up-front during loading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants