TelepathicGrunt / RepurposedStructures

Reusing and modifying vanilla structures for extra variety!
https://www.curseforge.com/minecraft/mc-mods/repurposed-structures
GNU Lesser General Public License v3.0
165 stars 26 forks source link

some problems i found #307

Closed peow closed 1 year ago

peow commented 1 year ago

first of all, great mod, im playing in fabric 1.20.1

1) RS structures can overlap vanilla structures 2023-10-22_23 56 22

here a RS mansion is generated in a vanilla village 2023-10-22_23 57 17

my fear with this is that some underground construction of RS will end up destroying a Stronghold portal in someone's world

2) RS structures can destroy bedrock

I found a mineshaft_nether that destroyed bedrocks underneath the nether 2023-10-23_14 25 48 2023-10-23_14 26 16

and as much as it may seem like it's not because of RS, I can confirm that it is, I tested the map seed using just the Fabric API, MidnightLib and Repurposed Structures

with RS: 2023-10-23_14 41 23

without RS: 2023-10-23_14 38 16

if you want to test it for yourself here is the seed and the coordinate:

seed: 2001851741016737369

tp: /execute in minecraft:the_nether run tp @s 356.86 -8.75 165.56 230.07 -43.92

the worst case is someone falling into the void and losing all their items

3) i think has to many piglins in nether's village or has another problem causing low fps 2023-10-24_22 38 50 58 fps

2023-10-24_22 39 00 191 fps after i killed 19 piglins

i made a test in a world with just Repurposed Structures and FPS Display mod

2023-10-24_22 48 38

2023-10-24_22 48 59

2023-10-24_22 49 06 increased almost 300 fps after I killed 38 nearby piglins

if you make an update pls do to fabric 1.20.1, tx

TelepathicGrunt commented 1 year ago
  1. This is intended and a limitation of the structure system. Vanilla woodland mansions can intersect their own villages. You can find seeds online about this so I won’t be doing any work to prevent this fusion. (Also I like fusion personally as they are rare anyway and something different to find)

  2. That bedrock breaking I’ll look at but I don’t see how it is rs. Unless it is a feature spawned from the structure rather than the structure pieces themselves.

  3. Piglin lowering fps is not a repurposed structures issue. That’s vanilla. You may see similar performance issue with vanilla bastions full of piglins and such but it is not game breaking and isn't unplayable tbh. If piglins are an issue in your modpack, verify you don’t have another mod causing lag with piglins and put on some rendering optimization mods if possible. The villages in nether spawns the same amount of piglins as vanilla villages spawn villagers and it is not within scope nor possible for repurposed structures to optimize piglin rendering without just removing piglins which I do not want to do. Entity rendering optimizing mods is what you'll want to use instead.

TelepathicGrunt commented 1 year ago

Also, no rs structure except for underground bastions even spawns on the same y level as strongholds and strongholds spawns after most structures so their end portal won’t be eaten away at

peow commented 1 year ago

i see, so the bedrock issue is the only problem

TelepathicGrunt commented 1 year ago

Bedrock issue fixed in v7.1.10. Issue wasn't the mineshafts but instead was the nether villages/outposts mainly. It has to do with "junctions" that connect the structure pieces together not being moved properly to the correct y value that rest of the structure is at.

Thank you for reporting that!

peow commented 1 year ago

i thank you for taking care and always updating your mod