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

changes ether's behaviour and recipe #15715

Merged
merged 25 commits into from Oct 28, 2023

Conversation

colossusqw
Copy link
Contributor

[LABEL][chemistry][feature][balance]

About the PR

This PR changes renames the ether chemical to Diethyl Ether, and restructures some of it's behaviour and the way in which it is made.

Behavior changes:

  • Diethyl Ether is extremely flammable, and besides combusting into a small fireflash at relatively low temperatures, also does it when in contact with any open flame.

  • Diethyl Ether now produces vapor when above 10c, making it continuously offgass at room temperature, which can be dangerous considering the vapor easily catches fire.

  • Diethyl Ether works slightly better as an anesthetic, having a higher overdose threshold(there is a big increase in difference between therapeutic and toxic doses), reducing the recent trauma effect, and having a 0.2 depletion unless currently KO-ing or above 10 units.

  • Diethyl Ether's KO moved to 40 cycles, but this can be reduced to 20 if more than 10 units present.

Recipe changes:

  • Base reagents(ethanol, hidrocloric acid and oxygen) remain the same, though the reaction only starts when above 30c.

  • Reaction now takes time, yielding Diethyl Ether at a base speed of 0.5 units per cycle, but that can increase with temperature. The reaction offgasses Diethyl Ether as well, but keeping the container closed causes it to cool the reaction, and requiring one to pay attention on heating to upkeep the reaction speed.

  • The reaction only happens while the amount of ether is inferior to the amount of ethanol present.

Why's this needed?

This gives the currently overlooked reagent Ether some more interesting effects, both in it's use in moderation as a easier to acquire anesthetic, and in it's possible use as a KO chemical. While it still is very much worse than capulettium in most regards, it now bridges the gap between it and morphine.

As per the reaction, Diethyl Ether's temperature based creation makes a compromise between how safe one wants to be and how expedient(and possibly effective) it's creation will be. It is easier to make than most other generic precursors, and if one keeps an eye on the reaction, it tends to go smoothly as well.

(u)Colossusqw
(*)Changed Diethyl Ether's base behavior and recipe(more info in minor changes).
(+)Diethyl Ether is now a time reaction that scales in speed with temperature, up until 80c, in which it becomes dangerously flammable. The reaction gives off fumes and only lasts as long as there is more ethanol than ether.
(+)Diethyl Ether now has a smaller depletion when below 10us and not currently KOing, if above 10 units, KOs in 20 cycles, else KOs in 40. It also helps reduce the recent trauma status effect.
(+)Diethyl Ether now gives off Ether vapor when above 10c.

[LABEL][AI][Chemistry][Balance]
## About the PR <!-- Describe the Pull Request here. What does it change? What other things could this impact? -->
This PR:
Makes it so rattlesnakes no longer bite borgs and chase after people if their personal space is intruded.
Changes the viper venom stagger effect for a 4 second stun.
Removes the random +8 tox chance.
Adds a small amount of bleed to complement the more dynamic take_bleeding_damage proc.

## Why's this needed? <!-- Describe why you think this should be added to the game. -->
With the advent of the new critter spawn event, i've noticed that encounters with rattlesnakes tend to be a lot more aggressive on the part of the snake than i'd like. This should make them more forgivable to be around.
As for the venom changes, the 10% chance of a +8 tox makes it wildly inconsistent, and stagger is rarely used in chemicals due to being mostly a combat effect, so i've changed those. As for the bleeding, a bump up on the bleeding speed should compensate for them no longer continuously biting you + the removal of the random extra damage.

## Changelog <!-- If necessary, put your changelog entry below. Otherwise, /please/ delete this entire section. -->

```changelog
(u)Hans
(+)Rattlesnakes will no longer chase people down unless directly attacked, they will still bite you if you get close.
```
@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 6, 2023
@pgmzeta pgmzeta added C-Feature A new feature or enhancements to existing features C-Balance Balance changes, buffs and nerfs A-Chemistry Deals with the chemistry system in some way labels Sep 11, 2023
Copy link
Contributor

@Flaborized Flaborized left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The design ideas/concepts look good to me, though I'll defer to another more experienced dev's review for the code. I like the changes though!

@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Sep 27, 2023
@colossusqw
Copy link
Contributor Author

!merge_upstream

@github-actions github-actions bot removed the S-Stale An inactive PR that has had no updates in the past two weeks label Sep 28, 2023
Copy link
Contributor

@TobleroneSwordfish TobleroneSwordfish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reaction appears to duplicate ethanol?
image
It can also spontaneously detonate due to offgassing heating the mixture for some reason.
Aside from that I doubt it will see much use due to having a relatively annoying recipe and mediocre effects, but that was already the case so I don't see harm in trying to make it more interesting.

code/modules/chemistry/Chemistry-Recipes.dm Outdated Show resolved Hide resolved
@TobleroneSwordfish TobleroneSwordfish merged commit 8dccd7d into goonstation:master Oct 28, 2023
21 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 28, 2023
@colossusqw colossusqw deleted the diethyl-ether branch November 28, 2023 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Chemistry Deals with the chemistry system in some way C-Balance Balance changes, buffs and nerfs C-Feature A new feature or enhancements to existing features size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants