Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@redmoogle
Copy link
Contributor

@redmoogle redmoogle commented Jul 27, 2022

Document the changes in your pull request

Upon taking 15 or more damage(IN A SECOND) you will get a small amount of adrenals (That just remove slowdown may add heart damage?)

Should last about 10 seconds
After receiving the effect it wont trigger for atleast 10 minutes

Should make combat slightly more fair since getting shot means you get stuck with movement slowdown which is basically instant lose

Open to balancing changes

Wiki Documentation

Combat takes slightly more effort now

Changelog

🆑
rscadd: Getting significantly hurt will remove slowdown for a few seconds
/:cl:

@Yogbot-13 Yogbot-13 added the Feature This adds new content to the game label Jul 27, 2022
@Hopekz
Copy link
Contributor

Hopekz commented Jul 27, 2022

Interesting PR

Also adds a tactical option of not attacking your attacker so you get the movement buff vs them to get away from them

@Marmio64
Copy link
Contributor

15 dmg sounds too low, too often it would be wasted getting hit on accident or something. Increase it to 25 or more. Also this may have severe balance issues on antags who rely on melee and damage slowdown, so I'm unsure how good this actually is for combat.

@redmoogle
Copy link
Contributor Author

15 dmg sounds too low, too often it would be wasted getting hit on accident or something. Increase it to 25 or more. Also this may have severe balance issues on antags who rely on melee and damage slowdown, so I'm unsure how good this actually is for combat.

conventional stuns and slowdowns still work

@Marmio64
Copy link
Contributor

15 dmg sounds too low, too often it would be wasted getting hit on accident or something. Increase it to 25 or more. Also this may have severe balance issues on antags who rely on melee and damage slowdown, so I'm unsure how good this actually is for combat.

conventional stuns and slowdowns still work

that still means you are restricting some antags to HAVING to use stuns or crafting bolas, while previously if you got the drop on someone as say heretic or cling you could manage to damage slowdown them in time before they can push you away and run, it was much riskier than using a stun but it gave you options while also giving the person being attacked an option to fight back (its a lot damn harder to fight against being bola'd/stunned and unable to do anything but talk)

@Chubbygummibear
Copy link
Contributor

would it be possible to make adrenaline tire the person out when it leaves their system. i'm pretty sure in real life people can be up for like days just riding on adrenaline and as soon as it stops they just collapse cause their body was doing 100% for the past whole time

Comment on lines 2065 to 2071
/datum/reagent/adrenaline/on_mob_add(mob/living/L)
. = ..()
ADD_TRAIT(L, TRAIT_IGNOREDAMAGESLOWDOWN, type)

/datum/reagent/adrenaline/on_mob_delete(mob/living/L)
. = ..()
REMOVE_TRAIT(L, TRAIT_IGNOREDAMAGESLOWDOWN, type)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
/datum/reagent/adrenaline/on_mob_add(mob/living/L)
. = ..()
ADD_TRAIT(L, TRAIT_IGNOREDAMAGESLOWDOWN, type)
/datum/reagent/adrenaline/on_mob_delete(mob/living/L)
. = ..()
REMOVE_TRAIT(L, TRAIT_IGNOREDAMAGESLOWDOWN, type)
/datum/reagent/adrenaline/on_mob_add(mob/living/L)
ADD_TRAIT(L, TRAIT_IGNOREDAMAGESLOWDOWN, type)
/datum/reagent/adrenaline/on_mob_delete(mob/living/L)
REMOVE_TRAIT(L, TRAIT_IGNOREDAMAGESLOWDOWN, type)

if you call parent it will jsut return

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tested this. it will just have no effect

@ynot01

This comment was marked as resolved.

@ynot01
Copy link
Contributor

ynot01 commented Jul 28, 2022

does stamina damage count

@redmoogle
Copy link
Contributor Author

does stamina damage count

does stamina damage count

for slowdown yes, but for triggering it, no

@Moggram
Copy link

Moggram commented Jul 28, 2022

I feel like this should be a human only thing. I understand why for balancing that everyone would get a adrenaline shot, but logically it makes no sense to me.

As Hopekz said, this is a interesting PR and like to see where it goes.

@Marmio64
Copy link
Contributor

I feel like this should be a human only thing. I understand why for balancing that everyone would get a adrenaline shot, but logically it makes no sense to me.

As Hopekz said, this is a interesting PR and like to see where it goes.

a human buff, and an interesting mechanic just for humans to get? No way the lizards and felinids would allow this to happen.

@tattax
Copy link
Contributor

tattax commented Jul 29, 2022

how does this affect miners
how does this affect melee combat
how does this affect stamina meta

@Hopekz
Copy link
Contributor

Hopekz commented Jul 29, 2022

how does this affect miners how does this affect melee combat how does this affect stamina meta

All those questions can be answered with a testmerge.

@Chubbygummibear
Copy link
Contributor

do bear in mind that if you take the human only route it will have other reaching effects like nukies benefiting from ignoring some degree of damage slowdown in combat. people would likely opt to stay human wizard too if it meant that getting tagged by someone wouldn't slow them so they could jaunt or whatever to escape more easily

@Skrem7
Copy link
Contributor

Skrem7 commented Jul 29, 2022

As soon as you give a roundstart species the ability to ignore base game mechanics with zero downside then you are opening a wild jar of balance that you don't want to spill

I like the thematic idea for making this human only but every other species would need a buff in return

@MegaEmpirical
Copy link

This is a cool idea.

Yogbot-13 added a commit that referenced this pull request Aug 5, 2022
Yogbot-13 added a commit that referenced this pull request Aug 28, 2022
Yogbot-13 added a commit that referenced this pull request Aug 29, 2022
Yogbot-13 added a commit that referenced this pull request Aug 29, 2022
Yogbot-13 added a commit that referenced this pull request Aug 29, 2022
@SegaCD SegaCD mentioned this pull request Aug 30, 2022
Yogbot-13 added a commit that referenced this pull request Aug 30, 2022
Yogbot-13 added a commit that referenced this pull request Sep 1, 2022
Copy link
Contributor

@Bibby0110 Bibby0110 left a comment

Choose a reason for hiding this comment

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

Seems fine

@Bibby0110 Bibby0110 merged commit fabd22d into yogstation13:master Sep 10, 2022
Yogbot-13 added a commit that referenced this pull request Sep 10, 2022
@redmoogle redmoogle deleted the combatschombat branch September 16, 2022 23:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Feature This adds new content to the game

Projects

None yet

Development

Successfully merging this pull request may close these issues.