-
Notifications
You must be signed in to change notification settings - Fork 6
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
Lag on first frame rendering Tinkers' Construct tools #67
Comments
Sorry if this isn't quite related to the issue, but you can see similar issues while searching through HEI. There aren't any texture issues in HEI anymore as far as I can tell. However, when HEI needs to render tinker's tools, depending on how many it needs to render, you can experience brief moments of freezing and the same types of errors are printed. I would assume it's because tinker's uses item model overrides and those aren't pre-baked by the dynamic resources functionality? |
Most likely. |
I also encountered this problem |
The issue is still actual and i would really appreciate fix! |
I assume most people experiencing this issue have CensoredASM installed. Does it help to disable |
For the test, i set |
I can also confirm that disabling releaseSpriteFramesCache through FermiumASM (ultimately a fork of CensoredASM) and Binkers Bonstruct does not reduce freezing from this error. Interestingly, reverting back to FoamFix removes the errors, but I'm still getting freezes slightly less, but still severe. The freezing on VintageFix is much longer. There are a significant amount of errors, the last of them in this log are related to the issue (with more detail) as I click through all the item pages on HEI. There is a crash that'll list all the mods a little ways up from the bottom. None of the errors restrict continuing to play. If you'd like an environment that'll generate this consistently, let me know! SlimeKnights/TinkersConstruct#3651 suggests this may be a long standing issue? Edit: As referenced in my latest.log above:
I set VintageFix's mixin.dynamic_resources to false and it's freezing behavior now mimics that of FoamFix where it's still severe, but not nearly as severe. |
In the latest beta I have applied many optimizations to both the Forge model baking pipeline and Tinkers itself. Please try it and let me know if the lag spike is less severe. (Please also let me know if the optimizations cause any models to render incorrectly compared to before.) https://nightly.link/embeddedt/VintageFix/workflows/gradle/main/Package.zip |
@embeddedt I got a crash when updating to the latest build |
Can you provide the full log please? |
Update: got a full log from another user with the same crash, and pushed a fix. |
Can confirm! Extraordinarily more manageable even with FermiumASM and Binkers Bonstruct. I wouldn't describe it as freezing at all, but minor hitching. I'll keep my eye out for any model rendering issues. Edit: I'm genuinely uncertain which changes that I've made have caused this, but Chisels and Bits are now exhibiting the same freezing behavior. I updated VintageFix to 0.4.2, then disabled it in curseforge and re-downloaded the above Package.zip. Oddly enough chisels and bits are set to not even show up in my JEI(HEI) list, but they are anyway, which might be more of an incompatibility with HEI. I'll keep testing to see if I can provide more clarity. Edit 2: Switching back to JEI removes the Chisels and Bits from JEI as intended which also removes the vintage errors regarding the model rendering. This would suggest that if Chisels and Bits were not hidden, the errors would persist. I still don't understand what the circumstances were that caused the game to be as smooth as it was when I originally posted this post. The log in this post does show VintageFix installed with the above package and still shows some errors, but I don't recall freezing. As far as Tinkers I don't see any freezing when scrolling past it's items. |
@embeddedt The latest version does fix the crash problem, but the lags when opening the tool forge does not seem to be fixed... |
The |
So what does the latest update actually fix? |
Item model loading in Forge & Tinkers was optimized to be faster. |
Tested the newest build, here's some spark results. Using With CensoredASM With CensoredASM Also, I would say in this minimal environment of Tinkers, CensoredASM, HEI, and VintageFix, I can no longer notice a slowdown when scrolling through the tinkers' materials and tools in HEI. |
I pushed some more minor improvements, but I don't know that they will be enough to suppress the delay. |
Just tested the new build, yeah it does seem roughly the same as when I tested in my previous post. I think the delay is pretty negligible though as long as CensoredASM's |
Is this still severe enough as of 0.5.0 to warrant further investigation, or can it be closed? |
Current lag is very insignificant and can be considered as fix. Thank you for the improvments! javaw_QvxLaAmTq6.mp4 |
Enable `dynamic_resources` config option of VintageFix. Enabling of this option could cause micro-second freezes when new items appear on screen. This performance improvment wasnt enabled since it caused ~30 seconds freeze when opening TConstruct Tool Forge. Now it was fixed embeddedt/VintageFix#67
Tested environment (all latest)
When mixin.dynamic_resources=true
mixin.dynamic_resources=false (Just like when you don't have this mod installed)
This has little effect in normal environments, but in large modpacks, it will cause a few seconds of freeze every time the Tool Forge GUI is opened
The text was updated successfully, but these errors were encountered: