Skip to content

/Cataclysm-DDA

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

Plant food boost. #6834

Merged
merged 3 commits into from Mar 23, 2014

Conversation

Reviewers
No reviews
Assignees

@KA101 KA101

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@Soyweiser @KA101
@Soyweiser
Copy link
Contributor

commented Mar 21, 2014

  • Plant food recipe gives 4 times the amount of plant food now.
  • Recipe requires 4 times the amount of water. (Reduces time spent on creating it, and reduces amount of chemicals used for the recipe).
  • Increased time reduction by plant food by 50% (was 2 days, is 3 now).
  • Changed the time reduction to give a 20% reduction depending on season length. On default this gives 2.8 days reduction.

As discussed in:
http://smf.cataclysmdda.com/index.php?topic=5823.0
@Soyweiser
- Plant food recipe gives 4 times the amount of plant food now. 
- Recipe requires 4 times the amount of water
- Increased time reduction by plant food by 50% (was 2 days, is 3 now).
Loading status checks…
2c9a3d3
src/iexamine.cpp Outdated
@@ -1195,7 +1195,8 @@ void iexamine::aggie_plant(player *p, map *m, int examx, int examy) {
}
} else if (m->furn(examx,examy) != f_plant_harvest && m->i_at(examx, examy).size() == 1 &&
p->charges_of("fertilizer_liquid") && query_yn(_("Fertilize plant"))) {
unsigned int fertilizerEpoch = 14400 * 2;
//Reduce the amount of time it takes until the next stage of the plant by 3 days.
unsigned int fertilizerEpoch = 14400 * 3;

This comment has been minimized.

Sign in to view
Copy link
@KA101

KA101 Mar 21, 2014

Contributor

Today in "no idea how difficult this is": would it be feasible to tie fertilizer to the season length? Under default seasons, fertilizer saved 1/7th of a season (2/14 days) per application.

Agreed that it's a pretty minimal bonus, though. :-/

This comment has been minimized.

Sign in to view
Copy link
@Soyweiser

Soyweiser Mar 21, 2014

Author Contributor

Hmm good one. Will have to look into that.
@Soyweiser

This comment has been minimized.

Sign in to view
Copy link
Contributor Author

commented Mar 22, 2014

I'll make it WIP till I fix the season thing.

@Soyweiser Soyweiser changed the title Plant food boost. [WIP] Plant food boost. Mar 22, 2014

Soyweiser added some commits Mar 23, 2014

@Soyweiser
Merge branch 'master' of https://github.com/CleverRaven/Cataclysm-DDA … 
…into plant_food
0295118
@Soyweiser
Fertilizer now gives a 20% reduction in time before the new stage. An… 
…d uses the currently set season length variable.
Loading status checks…
623930a

@Soyweiser Soyweiser changed the title [WIP] Plant food boost. Plant food boost. Mar 23, 2014

@Soyweiser

This comment has been minimized.

Sign in to view
Copy link
Contributor Author

commented Mar 23, 2014

Updated. Depends on season length now.

@KA101 KA101 self-assigned this Mar 23, 2014

@KA101 KA101 merged commit 623930a into CleverRaven:master Mar 23, 2014
1 check passed

1 check passed

default This has been rescheduled for testing as the 'master' branch has been updated.

@Soyweiser Soyweiser deleted the Soyweiser:plant_food branch Jul 28, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.