The-Aether-Team / The-Aether

The original "The Aether" mod, rewritten and ported to modern Minecraft versions.
https://modrinth.com/mod/aether
GNU Lesser General Public License v3.0
358 stars 116 forks source link

Bug: {Compatibility} 1.19.2 W/ Citadel's chunk gen spawn modifier set ≥4 #1825

Closed togera13 closed 11 months ago

togera13 commented 1 year ago

What Feature Types Apply to This Bug?

World-Gen

Other Type

No response

What Type of Bug Is This?

Compatibility

Forge Version

43.3.2 Forge

The Aether Version

1.0.0-beta.1.2-Forge

Is This Bug a Conflict With Another Mod?

citadel-2.1.4-1.19

Client Log

No response

Crash Report (if applicable)

No response

Steps to Reproduce

Open the citadel config set the chunk gen spawn modifier to anything equal to or higher than 4 load into a world place an Aether portal enter the Aether The Aether will not load & you will be endlessly falling, attempting to close the game will require task manager.

What You Expect To Happen

Load into The Aether.

What Actually Happened

Endless falling / server freeze / unclosable game (task manager needed).

Additional Details

https://www.curseforge.com/minecraft/mc-mods/aether/files/4771515 https://www.curseforge.com/minecraft/mc-mods/citadel/files/4556677

I was using Citadel's feature to increase the mob spawn count per chunk & the Aether would fail to load & freeze the server if anyone entered it when the config option of citadel was set to anything equal to or higher than 4.

Oddly enough it is only after it hits 4 that The Aether fails to load, 3 & below loads The Aether successfully, but as soon as the option is set to 4 or higher it fails to load The Aether.

Setting the option to 3 or lower seems to be a workaround-ish, I just wanted to make this known in case if anyone else was using the feature from Citadel to increase their mob spawns & was wondering why The Aether wouldn't load in.

Please Read and Confirm The Following

bconlon1 commented 1 year ago

I'll need the logs for this.

togera13 commented 1 year ago

This was all that was in the logs folder I don't know if this is what you needed or not, I ran it an additional time & posted the logs again so idk if one of them is a duplicate or not, if you have anything else you need me to do let me know. latest.log latest.log 2023-10-25-1.log

I appreciate your work btw.

bconlon1 commented 11 months ago

Source for reference: https://github.com/AlexModGuy/Citadel

bconlon1 commented 11 months ago

My current theory is that setting the value to 4 and quadrupling the spawn count is just making too many mobs spawn in the Aether for the game to handle, but I'm going to wait for a response from the Citadel developers since if it is only happening with Aether then that's still odd.

bconlon1 commented 11 months ago

This seems to technically be an issue with Citadel, they allow for the spawn modifier config value to be set too high for certain circumstances which causes stalls like this, so they'll need to add a safety check on their end. https://github.com/AlexModGuy/Citadel/issues/141