Thank you for looking into it, and i'm glad if this was useful
I agree that having to declare the list of nodes to apply restriction to, is very far from ideal...
As it is, it doesn't actually require any special configuration : if the server admin don't change the default config, it will dig uppward or on the same level for any kind of trees, but strictly uppward for jungle trees. But as said, I see your point and I understand why it doesn't fit.
I will keep thinking about how to do this in a better way, ... maybe by setting a radius for the orginal digged node or a limit to the number of node digged on the same level, this would probably work for any tree ...but this seems heavy too because that would mean keeping data on the first node pos accross all recusions.
Until then, I'm keeping a fork because I like being able to limit the "greediness" on my server but I've rebased it on your latest update
an option to only dig upward, leaving the bottom of the trunk (before | after).
an option to stop before the tool breaks
an option to even more restrict horizontal digging for some specific types of dense trees. This can prevents clear-cutting entire jungle portions, at the cost of maybe leaving some wood behind (before | after)
Hopefully it keeps the simple spirit of the mod, while adding these few features.
At least the code is way shorter than this comment :)
Each change is optional and come as a new setting disabled by default, so that the default behavior stays the same
Each is in a separate commit, and the last one adds gif screeshots
I don't have a github account to send PR but I thought you might want
to use it anyway.
Sorry I'm just seeing this now
Thank you for looking into it, and i'm glad if this was useful
I agree that having to declare the list of nodes to apply restriction to, is very far from ideal...
As it is, it doesn't actually require any special configuration : if the server admin don't change the default config, it will dig uppward or on the same level for any kind of trees, but strictly uppward for jungle trees. But as said, I see your point and I understand why it doesn't fit.
I will keep thinking about how to do this in a better way, ... maybe by setting a radius for the orginal digged node or a limit to the number of node digged on the same level, this would probably work for any tree ...but this seems heavy too because that would mean keeping data on the first node pos accross all recusions. Until then, I'm keeping a fork because I like being able to limit the "greediness" on my server but I've rebased it on your latest update
I didn't think about foliage, but once the tree has been cut, foliage disapear by itself
Having to dig manually to get the leaves seems fair to me
Thanks for the reply
The fork link was already in the initial comment :
About github, yes, as many, I left github when it was bought by microsoft :)
I like the simplicity of this mod ! ... but it works a bit too well for me, in the jungle I felt more like a buldozer than a lumberjack.
Also I use tool ranks mod and was very sad to see my level 8 axe break without warning
So I made a fork (here) that adds 3 things :
Hopefully it keeps the simple spirit of the mod, while adding these few features. At least the code is way shorter than this comment :)
Each change is optional and come as a new setting disabled by default, so that the default behavior stays the same
Each is in a separate commit, and the last one adds gif screeshots
I don't have a github account to send PR but I thought you might want to use it anyway.