samedifferent / morevillagers

🏠 A Minecraft mod that adds more villagers, trades, and workstations.
https://www.curseforge.com/minecraft/mc-mods/more-villagers
Other
40 stars 27 forks source link

[Bug]: Leveling Oceanographer to level 3 crashes world/server. #70

Open Gamemassa opened 1 year ago

Gamemassa commented 1 year ago

Mod Version

4.0.2

Mod Loader

Fabric

Mod Loader Version

Latest

What happened?

I leveled an Oceanographer up to level 3 in my friend's modded mp server (I can send the mod list if needed) and the server stalled for a minute before crashing. I tried replicating the bug in one of my sp worlds that I use to debug my friend's modpack for him, and my world stalled again, but after several minutes of waiting for it to crash (and it not crashing) I exited to main menu, reentered the sp world, and my oceanographer was level 3 (additionally I was able to level it the rest of the way from there with no issues).

latest.log

Here is the log from my friend's MP server: https://gist.github.com/Gamemassa/c87b54a6754b8f314f4b3876ec1051f8

And here is the latest.log from one of my SP worlds where I do testing on my friend's modpack: https://gist.github.com/Gamemassa/759b3e4c09ee6e9c079a2941b21679e2

samedifferent commented 1 year ago

The level 3 trade is a buried treasure map. You may have a mod that is preventing buried treasure from spawning.

Gamemassa commented 1 year ago

I don't think we have anything that would be causing that, but I suppose I could be wrong. Would you like me to send you our modpack so you can check (if you know how, because we're not sure) ?

-Edit: I just ran a search for buried treasure using Explorer's Compass (which is also in our modpack) and found one pretty quickly, so we definitely don't have anything preventing them from spawning.

2022-10-24_22 40 30

Gamemassa commented 1 year ago

The level 3 trade is a buried treasure map. You may have a mod that is preventing buried treasure from spawning.

Do you have any other ideas as to what might be happening?

samedifferent commented 1 year ago

Have you been able to reproduce the bug with no other mods installed?

Gamemassa commented 1 year ago

Have you been able to reproduce the bug with no other mods installed?

I just tried in one of my other instances which has almost no mods (except a minimap, shaders and a few other client-only GUI and QoL mods) and I didn't get the error this time, so it does seem like a mod conflict with something in my friend's modpack.

Out of curiosity, could it have something to do with Serilum's Infinite Trading mod? I know my friend has that in his modpack as well.

Gamemassa commented 1 year ago

Have you been able to reproduce the bug with no other mods installed?

Again, would you like me to send you a copy of my friend's modpack so you can take a look and see if anything jumps out at you?

Gamemassa commented 1 year ago

@samedifferent Out of curiosity, could I please get an update on this issue?

Gamemassa commented 1 year ago

Update: My friend actually managed to get our oceanographer up to level 5 last night. I'm not sure if the bug earlier was related to the fact that I'm not the server owner, but this may be something to look into.

chibranche commented 1 year ago

Same issue happened yesterday on my server.

Forge version, mod 4.0.3

Happened 3 times in a row

Gamemassa commented 1 year ago

Same issue happened yesterday on my server.

Forge version, mod 4.0.3

Happened 3 times in a row

Oh, the Forge version is doing that too? I think I have it in my custom Forge modpack but I hadn't tested that yet, so thanks for confirming.

chibranche commented 1 year ago

Here is the full modlist if that can help :

    server-1.19.2-20220805.130853-srg.jar             |Minecraft                     |minecraft                     |1.19.2              |DONE      |Manifest: NOSIGNATURE
    repurposed_structures_forge-6.3.5+1.19.2.jar      |Repurposed Structures         |repurposed_structures         |6.3.5+1.19.2        |DONE      |Manifest: NOSIGNATURE
    morevillagers-forge-1.19-4.0.3.jar                |More Villagers                |morevillagers                 |4.0.3               |DONE      |Manifest: NOSIGNATURE
    EnchantmentDescriptions-Forge-1.19.2-13.0.3.jar   |EnchantmentDescriptions       |enchdesc                      |13.0.3              |DONE      |Manifest: eb:c4:b1:67:8b:f9:0c:db:dc:4f:01:b1:8e:61:64:39:4c:10:85:0b:a6:c4:c7:48:f0:fa:95:f2:cb:08:3a:e5
    TerraBlender-forge-1.19.2-2.0.1.128.jar           |TerraBlender                  |terrablender                  |2.0.1.128           |DONE      |Manifest: NOSIGNATURE
    AmbientSounds_FORGE_v5.2.8_mc1.19.2.jar           |Ambient Sounds                |ambientsounds                 |5.2.8               |DONE      |Manifest: NOSIGNATURE
    trashslot-forge-1.19-12.0.0.jar                   |TrashSlot                     |trashslot                     |12.0.0              |DONE      |Manifest: NOSIGNATURE
    BiomesOPlenty-1.19.2-17.1.1.162.jar               |Biomes O' Plenty              |biomesoplenty                 |17.1.1.162          |DONE      |Manifest: NOSIGNATURE
    ToolStats-Forge-1.19.2-12.0.2.jar                 |ToolStats                     |toolstats                     |12.0.2              |DONE      |Manifest: eb:c4:b1:67:8b:f9:0c:db:dc:4f:01:b1:8e:61:64:39:4c:10:85:0b:a6:c4:c7:48:f0:fa:95:f2:cb:08:3a:e5
    projectvibrantjourneys-1.19.2-5.1.0.jar           |Project Vibrant Journeys      |projectvibrantjourneys        |1.19.2-5.1.0        |DONE      |Manifest: NOSIGNATURE
    dungeons_plus-1.19.2-1.2.2.jar                    |Dungeons Plus                 |dungeons_plus                 |1.2.2               |DONE      |Manifest: NOSIGNATURE
    CreativeCore_FORGE_v2.8.9_mc1.19.2.jar            |CreativeCore                  |creativecore                  |2.8.6               |DONE      |Manifest: NOSIGNATURE
    jei-1.19.2-forge-11.4.0.282.jar                   |Just Enough Items             |jei                           |11.4.0.282          |DONE      |Manifest: NOSIGNATURE
    AttributeFix-Forge-1.19.2-17.1.2.jar              |AttributeFix                  |attributefix                  |17.1.2              |DONE      |Manifest: eb:c4:b1:67:8b:f9:0c:db:dc:4f:01:b1:8e:61:64:39:4c:10:85:0b:a6:c4:c7:48:f0:fa:95:f2:cb:08:3a:e5
    DoggyTalentsNext-1.19.2-1.0.3.jar                 |Doggy Talents Next            |doggytalents                  |1.0.3               |DONE      |Manifest: NOSIGNATURE
    goblintraders-1.8.0-1.19.2.jar                    |Goblin Traders                |goblintraders                 |1.8.0               |DONE      |Manifest: NOSIGNATURE
    sophisticatedcore-1.19.2-0.5.15.142.jar           |Sophisticated Core            |sophisticatedcore             |1.19.2-0.5.15.142   |DONE      |Manifest: NOSIGNATURE
    gravestone-1.19.2-1.0.1.jar                       |Gravestone Mod                |gravestone                    |1.19.2-1.0.1        |DONE      |Manifest: NOSIGNATURE
    TerracartReloaded-forge-1.19.2-1.4.0.jar          |TerraCart Reloaded            |terracart                     |1.4.0               |DONE      |Manifest: NOSIGNATURE
    citadel-2.0.0-1.19.jar                            |Citadel                       |citadel                       |2.0.0               |DONE      |Manifest: NOSIGNATURE
    alexsmobs-1.20.2.jar                              |Alex's Mobs                   |alexsmobs                     |1.20.2              |DONE      |Manifest: NOSIGNATURE
    Bookshelf-Forge-1.19.2-16.1.11.jar                |Bookshelf                     |bookshelf                     |16.1.11             |DONE      |Manifest: eb:c4:b1:67:8b:f9:0c:db:dc:4f:01:b1:8e:61:64:39:4c:10:85:0b:a6:c4:c7:48:f0:fa:95:f2:cb:08:3a:e5
    sophisticatedbackpacks-1.19.2-3.18.29.719.jar     |Sophisticated Backpacks       |sophisticatedbackpacks        |1.19.2-3.18.29.719  |DONE      |Manifest: NOSIGNATURE
    guardvillagers-1.19.2-1.5.2.jar                   |Guard Villagers               |guardvillagers                |1.19.2-1.5.2        |DONE      |Manifest: NOSIGNATURE
    pmmo-1.19-0.5.5.jar                               |Project MMO                   |pmmo                          |1.19-0.5.5          |DONE      |Manifest: NOSIGNATURE
    architectury-6.3.49-forge.jar                     |Architectury                  |architectury                  |6.3.49              |DONE      |Manifest: NOSIGNATURE
    BambooEverything-forge-2.2.4-build.33+mc1.19.2.jar|Bamboo Everything             |bambooeverything              |2.2.4-build.33+mc1.1|DONE      |Manifest: NOSIGNATURE
    balm-4.5.3.jar                                    |Balm                          |balm                          |4.5.3               |DONE      |Manifest: NOSIGNATURE
    Enhanced_Celestials-forge-1.19.2-2.1.0.2.jar      |Enhanced Celestials           |enhancedcelestials            |2.1.0.2             |DONE      |Manifest: NOSIGNATURE
    CorgiLib-forge-1.19.2-1.0.0.6.jar                 |CorgiLib                      |corgilib                      |1.0.0.6             |DONE      |Manifest: NOSIGNATURE
    Oh_The_Biomes_You'll_Go-forge-1.19.2-2.0.0.13.jar |Oh The Biomes You'll Go       |byg                           |2.0.0.13            |DONE      |Manifest: NOSIGNATURE
    EnchantWithMob-1.19.2-6.6.0.jar                   |Enchant With Mob              |enchantwithmob                |1.19.2-6.6.0        |DONE      |Manifest: NOSIGNATURE
    forge-1.19.2-43.1.47-universal.jar                |Forge                         |forge                         |43.1.47             |DONE      |Manifest: 84:ce:76:e8:45:35:e4:0e:63:86:df:47:59:80:0f:67:6c:c1:5f:6e:5f:4d:b3:54:47:1a:9f:7f:ed:5e:f2:90
    FastLeafDecay-30.jar                              |FastLeafDecay                 |fastleafdecay                 |30                  |DONE      |Manifest: NOSIGNATURE
    xercamusic-1.19.2-1.0.0.jar                       |Xerca's Music Maker Mod       |xercamusic                    |1.19.2-1.0.0        |DONE      |Manifest: NOSIGNATURE
    structure_gel-1.19.2-2.7.1.jar                    |Structure Gel API             |structure_gel                 |2.7.1               |DONE      |Manifest: NOSIGNATURE
    DungeonsArise-1.19.2-2.1.52-beta.jar              |When Dungeons Arise           |dungeons_arise                |2.1.52-1.19.2       |DONE      |Manifest: NOSIGNATURE
Gamemassa commented 1 year ago

@chibranche For future reference, how did you export that mod list?

chibranche commented 1 year ago

I copy/pasted it from the crash log

Gamemassa commented 1 year ago

I copy/pasted it from the crash log

Oh right. You're using the Forge version, and the Forge crash logs display the mod list a bit differently.

Nic0las-Ponce commented 11 months ago

Same here, modded server, version 1.5.5, buried treasure maps exist adn have been used before

GamerEagle commented 10 months ago

Same thing happend to me, my game crash after leveling, is there any good trades after that or can i just play w/o using the oceanographer?

GamerEagle commented 10 months ago

Im using Modrinth and I was playing on a modpack called CrateOrigin that has this mod

Atakku commented 3 months ago

this is still an issue