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

Atomic drinks are not radioactive anymore[WIP] #39116

Merged
merged 1 commit into from
Mar 29, 2020

Conversation

Fris0uman
Copy link
Contributor

Summary

SUMMARY: Bugfixes "Atomic drinks are not radioactive anymore"

Purpose of change

Atomic coffee and atomic energy drinks actually irradiating player as if this game was fallout make no sense in vanilla.

Describe the solution

Change ATOMIC_CAFF iuse for CAFF and adjust the stim level to stay close of the previous effect on fatigue
Atomic coffee now "restores" 90 fatigue instead of 300 and bring at the brink of stimulant overdose
Atomic energy drink restores 102 fatigue instead of 324 and gives stimulant overdose of 4

Describe alternatives you've considered

Edit c++ code to remove radiation but keep the same effect on fatigue

Testing

Spawn the drinks
Drink
See effect on stimulant

Additional context

If this gets merge as is, iuse::atomic_caff won't be used by anything else in the repo so I don't know what should be done with it.

@Fris0uman Fris0uman changed the title Atomic drinks are not radioactive anymore Atomic drinks are not radioactive anymore[WIP] Mar 29, 2020
@kevingranade kevingranade merged commit 73bc329 into CleverRaven:master Mar 29, 2020
@Night-Pryanik
Copy link
Contributor

Is ATOMIC_CAFF used anywhere else?

twadams added a commit to twadams/Cataclysm-DDA that referenced this pull request Mar 30, 2020
@Fris0uman Fris0uman deleted the atomic_drink branch March 30, 2020 09:58
twadams added a commit to twadams/Cataclysm-DDA that referenced this pull request Mar 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants