Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign up[READY] Recipe reference rework #7814
Conversation
This comment has been minimized.
This comment has been minimized.
|
Why you hating on the oxford comma? |
This comment has been minimized.
This comment has been minimized.
|
Uh, yeah, here I thought you liked the Oxford comma. ??? |
KA101
reviewed
Jun 2, 2014
| recipes += "</color>"; | ||
| } | ||
| if(index == book->recipes.size() - 1) { | ||
| recipes += _(" and "); // Who gives a fuck about an oxford comma? |
This comment has been minimized.
This comment has been minimized.
KA101
reviewed
Jun 2, 2014
View changes
| @@ -9097,6 +9098,9 @@ void player::read(int pos) | |||
| return; | |||
| } | |||
| // otherwise do nothing as there's no associated skill | |||
| } else if (morale_level() < MIN_MORALE_READ && tmp->fun <= 0) { // See morale.h | |||
| add_msg(m_info, _("What's the point of reading? (Your morale is too low!)")); | |||
This comment has been minimized.
This comment has been minimized.
KA101
Jun 2, 2014
Contributor
Minor nitpick: maybe change this to "What's the point of studying?" or somesuch might be better, since reading fun stuff can still work?
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
KA101
reviewed
Jun 2, 2014
View changes
| if(g->u.knows_recipe(iter->first)) { | ||
| recipes += "</color>"; | ||
| if (book->req == 0) { | ||
| dump->push_back(iteminfo("BOOK", _("It can be understood by beginners."))); |
This comment has been minimized.
This comment has been minimized.
KA101
Jun 2, 2014
Contributor
Thinking that beginner-level books will probably be somewhat obvious from their cover.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Nitpicks notwithstanding, I like the general gist here. Looking forward to readiness & mergetest. |
This comment has been minimized.
This comment has been minimized.
|
Joking a bit about the comma thing, quoting a song. I do think it's icky, it's redundant with the "and" right next to it. |
This comment has been minimized.
This comment has been minimized.
|
Tsk tsk. Remember the effect it can have on JFK and Stalin. http://stephentall.org/2011/09/19/oxford-comma/ [EDIT: slightly not safe for work, but only very slightly.] |
This comment has been minimized.
This comment has been minimized.
|
I like this. I've been teaching for the last 3 years, and it amazes me how people think they can learn by sitting back and "absorbing" the code from the screen. Big plus for learn-by-doing encouragement. I'd rather potentially waste materials prototyping vs. the grind of reading before I can do. |
This comment has been minimized.
This comment has been minimized.
|
Linking to Issue #7772. |
This comment has been minimized.
This comment has been minimized.
|
Being totally self-taught, in every way, having spent my schooldays mostly running for my life. |
This comment has been minimized.
This comment has been minimized.
|
"We invited the stripper, Jackie O, and Stalin." |
This comment has been minimized.
This comment has been minimized.
|
Uh, looking at that table there, I'm thinking the axes are flipped, or I'm missing something? If I'm reading it right, it'd take me 10K minutes of making a level 1 recipe to memorize it, if I had rank 10 in the relevant skill. That seems off. |
This comment has been minimized.
This comment has been minimized.
|
I'm not sure if I worded it right or not, difficulty goes from low to high as you go right, skill increases as you go down. So that extreme upper-right is 10 difficulty and 1 skill, which can't actually happen, but I was in a hurry to finish it, so I didn't trim the impossible results. |
This comment has been minimized.
This comment has been minimized.
OK, yeah, that makes more sense. Thanks. As for memorizing from the book, perhaps halve the base chance, have Good Memory double it back to the current rate, and Forgetful either make it impossible or reduce the chance further? (Thinking those traits need a bit more love, since skill rust is not universally accepted.) |
kevingranade
changed the title
[WIP] Recipe reference rework
[MAYBE READY] Recipe reference rework
Jun 6, 2014
This comment has been minimized.
This comment has been minimized.
|
That commit is in there removing memorization from reading, but it's trivial to yank it back out. |
This comment has been minimized.
This comment has been minimized.
|
Here's a better idea, halving the learn rate, and PRing removal of learning recipes by reading seperately. |
kevingranade
changed the title
[MAYBE READY] Recipe reference rework
[READY] Recipe reference rework
Jun 6, 2014
kevingranade
referenced this pull request
Jun 6, 2014
Closed
[DISCUSS] Disable recipe memorization from books #7866
This comment has been minimized.
This comment has been minimized.
|
In terms of knowing recipes, are recipes random? I like the disabling memorization from reading books, +1 for me on that (which has landed in your other PR). From a UI perspective, when I'm tossing houses and looking for reading material, if I examine a book but don't read it, will I get a hint that there are recipes in the book? I didn't see it in the code, it's probably there, but I just missed it. |
This comment has been minimized.
This comment has been minimized.
|
I don't know what you mean by "knowing" recipes, if you mean the way it works now, yea it's randomized which recipe you learn when reading a book that has multiple recipes in it. No, you'll need to spend some time reading it to find that out, though if that time period is unreasonable (I honestly haven't checked it), I'd be open to making the first read a shorter action that only gets you that information. (a minute or so? reasonable to do while looting a house, but not in combat) |
This comment has been minimized.
This comment has been minimized.
|
My first sentence above should have been deleted before sending. Ignore the "knowing" part, but thank you for attempting to answer it. Is this the proposed flow?
Given the implicit id by title, i'm not sure the read to discover the recipe adds a lot to the gameplay experience unless recipes start getting randomized dynamically. If recipes do get randomized, then the required reading to id the recipes would be a fun addition. Nethack has id scrolls, but here we read things to id them an discover recipes. I might start picking up those Spy Novels if one might randomly have a recipe in it. |
This comment has been minimized.
This comment has been minimized.
|
You are correct about the flow of how it's intended to work, though |
This comment has been minimized.
This comment has been minimized.
Clarify: so skill-from-book increases half as fast, or you get recipes from books (assuming for this question that the other PR doesn't exist) half as often, or some other meaning I'm not getting? Some skills are enough of a pain to raise via books already, and slowing that down even more seems a bad idea IMO. |
This comment has been minimized.
This comment has been minimized.
|
I can get the immersion thing, especially since it's appropriate. We have To KA101s point, I have a pretty uber Krav Maga character turned Farmer/Homesteader. My guy kicks arse for the lord and doesn't need a lawn mower, but would rather grow wheat (which he just found some seeds of). But... some of his skills are a bit pathetic. Traps is a decent example of a skill that might need some love. I can't raise traps for the life of me. I don't need them, but I want them and I feel stuck and don't really want to grind learning traps. My dodge is also uber low. I think it's at 1.5. I've even stood in front of Zombie Children to up it, but man that skill takes a while and I went back to whapping anything green on my screen. On the other end of the spectrum, survival definitely seems a bit unbalanced. I've gained quite a bit of skill just butchering zombies. Summary:
|
This comment has been minimized.
This comment has been minimized.
|
Gagh, missed the commit message there. I think this looks ready for testing, yeah. |
KA101
self-assigned this
Jun 7, 2014
This comment has been minimized.
This comment has been minimized.
|
Tested. After my snag-fix, I'm getting some cannibal recipes bleeding through (because tio tacos and manwich are autolearned at 3, apparently. Hmm), and the initial read cycle goes until you learn a recipe (for some of the technical documents, you're looking at 45 min or so). Suggest halving/quartering the read time for the ID-read, or perhaps pegging it to X-INT minutes. Should probably insert a line somewhere "You reference your $BOOK" so players know why their recipes disappear when they leave the book on the other side of the evac shelter. (They're parsed as a crafting ingredient, so the 6-tile radius may take people by surprise.) Was able to learn raptor serum on the first batch had Cooking 10, cocktail serums require 10 to make. Same for alpha serum...and both take 15K minutes to make, so that's within parameters, but might blow the model. (Same effect with poppy stuff and aspirin.) Thoughts? |
KA101
reviewed
Jun 7, 2014
View changes
| @@ -1053,7 +1067,7 @@ nc_color item::color(player *u) const | |||
| ret = c_green; | |||
| } | |||
| } | |||
| } else if (is_book()) { | |||
| } else if (is_book() && u.has_identified( type->id ) ) { | |||
This comment has been minimized.
This comment has been minimized.
KA101
Jun 7, 2014
Contributor
Snag-fix: Code::Blocks wanted a pointer, not a period. :-/ Sorry, Kevin.
kevingranade
added some commits
Jun 2, 2014
This comment has been minimized.
This comment has been minimized.
|
I can't reproduce either issue after a rebase, pretty sure the flu shot thing was an upstream issue, and I don't know about reading in the dark, do you maybe have a really dim light source on you, like a lightstrip? |
This comment has been minimized.
This comment has been minimized.
Rivet-the-Zombie
added a commit
that referenced
this pull request
Jun 9, 2014
Rivet-the-Zombie
merged commit 0dbb6e6
into
CleverRaven:master
Jun 9, 2014
This comment has been minimized.
This comment has been minimized.
|
You can read in dark, it takes up to 5x as long. The new warning message in the screenshot about being difficult to see fine details is to let you know it's poor light for reading. You shouldn't be able to read in "very dark," though, like in a basement with no light source. In my testing it seems to still correctly disallow that with "You can't see to read!" |
This comment has been minimized.
This comment has been minimized.
|
Thanks rluddy. My ignorance. I think you were the one who checked in that warning message? Thanks for that. Little things like that help me a lot. |
This comment has been minimized.
This comment has been minimized.
3t3rn4lfl4m3
commented
Jun 10, 2014
|
When I try to make cooked creep referring to my cookbook, I get floating point exceptions. If I debug to learn the recipe, I have no problem making it. Version: 0.A-3507-gbead0b2 |
This comment has been minimized.
This comment has been minimized.
|
I would suggest making it clear in the messages that a lack of light is why it is difficult to read/craft. "It's too dark, making it difficult to see fine details." "Without better lighting, reading will take much longer than usual." This would clue in newer players, as many other games let you read in pitch darkness, even though it is completely unrealistic. |
This comment has been minimized.
This comment has been minimized.
Uh, doesn't it do that already? |
This comment has been minimized.
This comment has been minimized.
|
Looking at the screen shot above it doesn't mention WHY it is hard to see. The words "light" or "dark" are not in the messages. It may be obvious to some. Maybe not to others. Especially when it is daytime and the tiles around the player are revealed. |


kevingranade commentedJun 2, 2014
X Remove/lower chance to memorize recipe from reading book. (punted to another PR)
This is the current number of minutes of crafting required to learn a recipe based on skill(rows) and difficulty(columns):