Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

small poplar trees seed not to exist anymore #17

Closed
fluxionary opened this issue Nov 5, 2022 · 6 comments
Closed

small poplar trees seed not to exist anymore #17

fluxionary opened this issue Nov 5, 2022 · 6 comments
Labels
bug Something isn't working

Comments

@fluxionary
Copy link
Member

all poplar saplings grow into the same size (ongen included). i'm pretty sure it wasn't always this way.

also, the icon for the small poplar ongen sapling is incorrect.

@Niklp09 Niklp09 added the bug Something isn't working label Nov 5, 2022
@Niklp09
Copy link
Member

Niklp09 commented Nov 5, 2022

confirmed

@S-S-X
Copy link
Member

S-S-X commented Nov 5, 2022

Yeah, it seemed to be planned change but I didn't like it.
100% support for restoring small poplars and also keeping old challenges for making those small poplars actually grow.
At least it was different in many ways which actually did make it very special.

@wsor4035
Copy link
Contributor

wsor4035 commented Nov 6, 2022

assuming s/seed/sapling from the title, it looks like when placed the small varient is getting converted to a large one by an abm/lbm perhaps. as for the icon, it is only incorrect for the fast growth/on gen one

@wsor4035
Copy link
Contributor

wsor4035 commented Nov 6, 2022

some weirdness happening here https://github.com/mt-mods/moretrees/blob/master/node_defs.lua#L567-L590 if i comment out the register_node's, it breaks the non fast growth, and removes the texture from the fast growth, but it the small fast growth does grow properly. have to try and figure out what is up with this workaround code

edit: the poplar small regular sapling isnt registered because there is an if statement which looks like its supposed to prevent the wood/stairs/etc from getting registered for defaults nodes. aka it looks like poplar got stuck in here rather than getting its own exception

@wsor4035
Copy link
Contributor

wsor4035 commented Nov 6, 2022

ok, will be pushing a fix to master shortly for this. issue caused by existing hacky workarounds

@wsor4035
Copy link
Contributor

wsor4035 commented Nov 6, 2022

tldr the issue was caused by a hacky workaround of including the small variant in the same exception for the default trees, so i removed it from there, and properly supported in the regular registration while removing the old hacky workarounds

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants