Micalobia / Full-Slabs

Adds vertical and mixed slabs to Minecraft
GNU General Public License v3.0
8 stars 2 forks source link

Full Slabs 3.1.3 causes massive loading times when used in a large modpack #38

Closed SplendidAlakey closed 3 years ago

SplendidAlakey commented 3 years ago

MC 1.17.1 FullSlabs 3.1.3

When FullSlabs 3.1.3 is installed, it takes 10 minutes to load the game. Does not happen without FullSlabs and also didn't happen with FullSlabs 3.1.2. Also does not happen when 3.1.3 is used alone. Modlist can be found in both latest.log files, however, I can provide it separately, if needed.

Latest.log without FullSlabs: [removed]

Latest.log with FullSlabs: [removed]

Since FullSlabs works fine on its own, I'll understand if this issue will not be fixed from your side.

Micalobia commented 3 years ago

If it doesn't happen with 3.1.2, then it has to do with the fix I made for the server-client desync in 3.1.3. I'll look into it, I don't expect it to be too difficult to fix

Basically, it's sorting a list every time a slab is added outside of vanilla, which I didn't think about the performance implications of. The fix should just be making it sort the list after all mods have loaded

SplendidAlakey commented 3 years ago

Just to be absolutely sure, I tested 3.1.2 again and yes, it loads normally.

Here's the log if it's of any use: [removed]

Thanks for looking into it!

Micalobia commented 3 years ago

So there's a new version called 3.1.4.exp1, which should fix this issue but I'm not sure. If you could test it I would be grateful.

SplendidAlakey commented 3 years ago

Just tested it, the fix worked. Loaded in just 2 minutes, which is only like ~20 seconds longer than with 3.1.2, at which point it might just be a margin of error.

Thanks again!

SplendidAlakey commented 3 years ago

Upon further testing, it seems like loading times moved into the worldgen. It seems a bit faster anyway, though. When I loaded up my world it took 5 minutes to load. Tested it on a server as well, it loaded fast, but 50% of the blocks are invisible/wrong.

Client log: [removed]

What happens when joining a server:

2021-10-02_16 00 45

What's weird is that most of it should be stone bricks, yet only some are invisible.

And this is how it should look like:

2021-10-02_16 11 05

Micalobia commented 3 years ago

Are you using a mod pack that you've built off of, or is this just something you put together? Going to attempt to download it myself, just want to make sure 😄

Micalobia commented 3 years ago

Also thank you for being patient and continuing to reply, it's extremely helpful

SplendidAlakey commented 3 years ago

I put it together myself. You will need to edit MineGate config in order to be able to create worlds with this modpack, in its ini files or modmenu change "useMixins" to false. Other than that I don't think any of my custom configs matter, but, if needed, I can provide all of them.

And no problem at all, I love your mod, it's the best slab mod I've seen and I'd love to include it in my modpack.

SplendidAlakey commented 3 years ago

These 2 modpacks are the closest to what I have: BetterMinecraft and All of Fabric 4. But they don't have all the mods I do, and I don't use some mods that they include.

Micalobia commented 3 years ago

I've released a new experimental version if you could take a look

Thank you for reminding me to give explicit permission to use in modpacks, btw, that's a good idea

SplendidAlakey commented 3 years ago

The second experimental version works flawlessly so far. Initial start at about 2.5min, single-player world loaded in just ~30s. While connected to a server all blocks appear as intended. Amazing job! I will be keeping an eye out for any issues down the line, but this one can be considered fixed.

Micalobia commented 3 years ago

Cool beans, thank you for reporting!

SplendidAlakey commented 3 years ago

Ignore that, it was caused by some other mod (I think Eating animation).

Oh, just 1 thing I forgot to mention (not related to this issue at all, so let me know if it's worth opening another one for it) - Cut copper slab doesn't have the HUD elements when it's placed as a horizontal slab:

2021-10-02_19 56 25

2021-10-02_19 56 28

I don't know if this is an issue with the experimental version or just in general with the mod.