Choppy ignores parts of tree #4656

New Issue

niceride · 2023-06-01T16:14:18Z

niceride commented

2023-06-01 16:14:18 +00:00

Choppy does not see the node where the sapling sprouted into its tree form, when chopping this it doesn't reliably start choppy and chopping Y+1 from the sprout node doesn't always chop both up and down, so ending up with stumps.

niceride changed title from ~~Choppy ignores sprout node~~ to Choppy ignores parts of tree

2023-06-01 16:18:41 +00:00

Boot commented

2023-06-01 16:45:35 +00:00

If you aim at the lower block at the start, the whole tree will be cut down. So I don't see a problem.

flux added the

1. kind/bug

3. source/mod upstream

labels 2023-06-01 19:15:07 +00:00

flux commented

2023-06-01 19:17:33 +00:00

if you turn the tree into two or more disconnected parts, there's not a whole lot choppy can do. if it's just the stump, it's also trivial for the player to just cut it down themselves. i'm not sure this is worth any sort of "fix".

one thing i've been meaning to do, is to not have choppy stop the process when the tool wears out, and instead "pause" and wait for the player to repair the tool.

if you turn the tree into two or more disconnected parts, there's not a whole lot choppy can do. if it's just the stump, it's also trivial for the player to just cut it down themselves. i'm not sure this is worth any sort of "fix". one thing i've been meaning to do, is to not have choppy stop the process when the tool wears out, and instead "pause" and wait for the player to repair the tool.

👍 2

niceride commented

2023-06-02 20:16:49 +00:00

How is it that woodcutting got this (mostly) right? Or I am remembering poorly.

flux commented

2023-06-02 20:28:22 +00:00

How is it that woodcutting got this (mostly) right? Or I am remembering poorly.

woodcutting didn't start at the top of the tree and go down.

i've figured out a way to solve this though: 348b252253

> How is it that woodcutting got this (mostly) right? Or I am remembering poorly. woodcutting didn't start at the top of the tree and go down. i've figured out a way to solve this though: https://github.com/fluxionary/minetest-choppy/commit/348b2522532c52833e373405c821dbcf35c552df

👍 1

flux added the

4. step/ready to QA test

label 2023-06-02 20:28:28 +00:00

AliasAlreadyTaken added this to the 1.1.122 milestone 2023-12-06 13:52:51 +00:00

AliasAlreadyTaken added the

ugh/QA OK

label 2023-12-06 13:52:56 +00:00

AliasAlreadyTaken commented

2023-12-06 13:53:20 +00:00

QA

Most likely in since some updates, but only now in QA