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.

Sign up for GitHub

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

Jump to bottom

plump_helmet_grow_time may not be correct #18

Closed
LeafyLappa opened this issue Jan 12, 2021 · 1 comment
Closed

plump_helmet_grow_time may not be correct #18

LeafyLappa opened this issue Jan 12, 2021 · 1 comment

Comments

@LeafyLappa
Copy link

There seems to be a typo in how plump_helmet_grow_time is calculated in this line in plump_helmet.lua:

local plump_helmet_grow_time = df_farming.config.plant_growth_time * df_farming.config.cave_wheat_delay_multiplier / 4

Quick comparison against other plants suggests plump_helmet_delay_multiplier should be there and not cave_wheat_delay_multiplier.
FaceDeer added a commit that referenced this issue Jul 15, 2022
@FaceDeer
fix typo in grow time, this was using cave wheat's delay multiplier
a243b61 
#18
@FaceDeer
Copy link
Owner

Quite right, thanks for catching that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@FaceDeer @LeafyLappa