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

Chemical Revolution: Chemistry Skill and Lithium #38005

Closed

Conversation

roaringjohn
Copy link
Contributor

Summary

SUMMARY: Content "Separate Chemistry from Cooking and add Lithium reaction recipes"

Purpose of change

To start the expansion of chemistry in Cataclysm.

Describe the solution

As the base of any and all Chemical Revolution PRs I make, I separated the cooking skill into cooking and chemistry and plan to add lithium and lithium reactions as the first step.

Describe alternatives you've considered

For the skill separation? Proficiencies as mentioned in the Practice makes Perfect project. For the lithium stuff? I dunno.

Testing

Find lithium in labs, on scientists, etc. Make chemistry recipes to advance chemistry skill.

Additional context

First part of my "Chemical Revolution" series of PRs.

Because the "Practice makes Perfect' project has barely been touched, I decided that I'd go ahead with separating the Cooking skill into two different skills.
Change some recipes to take into account the new recipe book I added, and added a few lithium items as well as hydrofluoric acid. Still not in itemgroups, nor do they have recipes. That will be done in the next commit.
@NastyNate2612
Copy link
Contributor

Lol i just put out my extracts and concentrates data the other day. whenever this gets added ill have to go and change the skill requirements for all the recipes for it

@ZhilkinSerg ZhilkinSerg added 0.E Feature Freeze <Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling Mechanics: Effects / Skills / Stats Effects / Skills / Stats labels Feb 14, 2020
For later commits and later PRs, I decided to add a lower-level organic chemistry textbook to go with the lower-level inorganic chemistry textbook I already added. Might break this up if suggested.
Added a "high school"-level chemistry textbook for levels 0 through 2.
@roaringjohn
Copy link
Contributor Author

For personal reference after I return on Sunday:
Search through the recipes for tools that lean towards a chemistry focus to apply the chemistry skill to it if necessary.

@roaringjohn
Copy link
Contributor Author

roaringjohn commented Feb 14, 2020

Lol i just put out my extracts and concentrates data the other day. whenever this gets added ill have to go and change the skill requirements for all the recipes for it

If your PRs get merged first, I'll make sure to make a new PR to take your changes into account.

Just one more major lithium compound. Will be going on to adding recipes on Sunday, along with taking another look through recipes to replace cooking with chemistry depending on the tools used.
@ZhilkinSerg
Copy link
Contributor

Feel free to reopen if you want working on it again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Crafting / Construction / Recipes Includes: Uncrafting / Disassembling <Enhancement / Feature> New features, or enhancements on existing [JSON] Changes (can be) made in JSON Mechanics: Effects / Skills / Stats Effects / Skills / Stats stale Closed for lack of activity, but still valid.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants