ValkyrienSkies / Eureka

Basically Archimedes Ships, but with VS2 as the backend
Apache License 2.0
54 stars 32 forks source link

minecraft:snow_block and possible minecraft:light need to be added to the blocklist #248

Closed spottenn closed 9 months ago

spottenn commented 10 months ago

This issue occurs when only Valkyrien Skies and addons are installed and no other mods

Minecraft Version

1.20​

Mod Loader

Fabric

Issue description

When in snowy slopes and similar, if your ship accidentally touches a snow block and you hit asssembly, all the connected snow blocks also get assembled into the ship. We accidentally made a giant mountain range sized ship with this. I'm a developer and I looked through the code, The correct file to add it to is "Eureka/common/src/main/kotlin/org/valkyrienskies/eureka /EurekaConfig.kt"

Also, the light block should probably be added because the user doesn't expect the ship to include light blocks and our shipping was touching the snow blocks via a light block.

Issue reproduction

Go to any biome with many adjacent snow blocks (not just snow, but full snow blocks) and stick a helm on one of blocks and look at the hitbox size of the ship (f3+b).

Logs

No response

NullHarp commented 10 months ago

technically not a bug, just not a desirable outcome, you can add snow manually to the blacklist by editing the config

NullHarp commented 10 months ago

also light cant be assembled

millennIumAMbiguity commented 10 months ago

Since Snow is available in all versions, the blacklist would need to be updated on all versions. Since 1.20 is behind 1.19, as long as snow is added to 1.16, 1.18, and 1.19; all versions will In the future have the correct blacklist.

You said that you are a developer yourself, feel free to make a pull request.

millennIumAMbiguity commented 9 months ago

merged to main