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

A kit to help new botanists discover chemistry #31738

Merged
merged 6 commits into from
Sep 5, 2024

Conversation

drakewill-CRL
Copy link
Contributor

About the PR

Added the "Agrichem is Fun!" kit to botany's Nutrimax vendor. It's a box containing 30u of several chemicals to make basic fertilizers, and 5u of 2 others to make unstable mutagen. It's meant to be reminiscent of the old 1950s chemistry kits that had uranium or other dangerous chemicals in them, and useful for new botanists.

Why / Balance

Hydroponics is a pretty chill department, but there's very little in-game information or training on anything there. A lot of learning comes from the wiki or other players. This change is done to provide something in-game that shows new botanists there's more to the job than "plant seed in tray, harvest produce later". It also introduces them to the game's chemistry system with some good chemicals they will want and an explanation of what they do.

Yes, all this information is in the guidebook. No, not everyone is going to read that. This is a way to find some of that out in-game without reading the OOC guidebook or wiki or such.

This kit is pretty limited, both in amount of reagents and products you can make with them. The idea is to get more of the people learning botany to look at the next step in complexity on the job, since there's not much right now to help you discover that on your own. The kit shows you how to make more of the baseline chemicals you use, along with alternative chemicals with stronger effects, and a little bit (15u) of robust mutagen without bugging chemistry.

Technical details

This is all YAML, no C#.

Media

Warning: codersprites

image

Requirements

  • I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

N/A

Changelog

🆑

  • add: Botanists can now learn basic chemistry with the Agrichem Is Fun! kit in the Nutrimax vendor.

… Intended to help new botanists learn in-game about chemistry.
@github-actions github-actions bot added Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Sprites Changes: Might require knowledge of spriting or visual design. labels Sep 2, 2024
Copy link
Contributor

github-actions bot commented Sep 2, 2024

RSI Diff Bot; head commit 5074504 merging into 10877eb
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Objects/Storage/boxes.rsi

State Old New Status
agrichemkit Added

Edit: diff updated after 5074504

@Emisse
Copy link
Contributor

Emisse commented Sep 2, 2024

this is cute

@github-actions github-actions bot added the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Sep 2, 2024
Copy link
Contributor

github-actions bot commented Sep 2, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the S: Merge Conflict Status: Needs to resolve merge conflicts before it can be accepted label Sep 2, 2024
@slarticodefast slarticodefast added the S: Undergoing Maintainer Discussion Status: Currently going through an extended discussion amongst maintainers, as per procedure. label Sep 2, 2024
@Plykiya
Copy link
Contributor

Plykiya commented Sep 2, 2024

mandatory feature

@slarticodefast
Copy link
Member

This idea is a great way to introduce a fun mini-tutorial for newer players.
I brought it up for maintainer discussion and before merging we would request you to change the name "farmer's helper" to unstable mutagen to avoid any confusion when looking at the guidebook and talking to chemists. You may have to rewrite the paragraph for that a little.

@slarticodefast
Copy link
Member

These are so much fun to read. You are great at writing :)
Thank you for your contribution!

@slarticodefast slarticodefast merged commit 511d759 into space-wizards:master Sep 5, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Sprites Changes: Might require knowledge of spriting or visual design. S: Undergoing Maintainer Discussion Status: Currently going through an extended discussion amongst maintainers, as per procedure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants