CleverNucleus / data-attributes

Minecraft Fabric mod that overhauls the entity attributes system and exposes it with datapacks.
MIT License
12 stars 15 forks source link

server side crash because of mansion #96

Closed dadoirie closed 1 year ago

dadoirie commented 1 year ago

as only Data Attributes is mentioned I'll report it meanwhile, but it might be most likely a conflict here - which I'm still trying to figure out, but with the amount of mods this might be taking a while (but if having suspicion it would be real helpful)

after removing one structures mod the locating did not crash anymore (which happened 3 times with that structures mod removed) after removing that structure mod locating when fine, just when teleporting to that coordinates of the minecraft:mansion server crashes

crashlog: https://mclo.gs/OW5NYw0

after removing PlayerEx, RelicEx, WizardEx & Data Attributes no such crashed occurred after teleporting to the mansion will investigate further and see which of those Ex's are causing the crash and if it's one of them I will move this issue to the appropriate Github

dadoirie commented 1 year ago

found the conflict: Illager Expansion [Rematch]

log file: https://mclo.gs/HaF7Ndo

UPDATE: has no hurry since this mod has other issues besides that one and therefore not using it anymore - the crash also might be not even data attributes fault at all

CleverNucleus commented 1 year ago

Hi there, I am aware of the issue - it is to do with the unconventional way Illager Expansion uses to register attributes. I have made a PR on their github with an easy-to-implement fix and therefore consider this matter closed.

Regardless, I appreciate you taking the time to troubleshoot the issue. Thanks.

dadoirie commented 1 year ago

thanks a lot for that PR