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

[MIRROR] unstashable assembly bombs #2026

Merged
merged 1 commit into from
Feb 17, 2024
Merged

[MIRROR] unstashable assembly bombs #2026

merged 1 commit into from
Feb 17, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#998
Original PR: tgstation/tgstation#81197

About The Pull Request

assembly bombs are bulky
(the like ones you make with a tank and an igniter and some other thing)

Why It's Good For The Game

these things can (and usually do) gib people, are REALLY easy to massproduce and you can also fit several in a bag
while still easier to use to gib people than TTVs

at best i think making them bulky should prevent some dude from carrying like 8 bombs in his backpack and whipping them out and throwing without any warning because they would have to carry it (or invest in BoH but thats sci anyway so)

Changelog

🆑 mc-oofert
balance: assemblybombs are bulky
/:cl:

* unstashable assembly bombs (#81197)

## About The Pull Request
assembly bombs are bulky
(the like ones you make with a tank and an igniter and some other thing)

## Why It's Good For The Game

these things can (and usually do) gib people, are REALLY easy to
massproduce and you can also fit several in a bag
while still easier to use to gib people than TTVs

at best i think making them bulky should prevent some dude from carrying
like 8 bombs in his backpack and whipping them out and throwing without
any warning because they would have to carry it (or invest in BoH but
thats sci anyway so)

## Changelog
:cl:
balance: assemblybombs are bulky
/:cl:

---------

Co-authored-by: tattle <[email protected]>

* unstashable assembly bombs

---------

Co-authored-by: jimmyl <[email protected]>
Co-authored-by: tattle <[email protected]>
@Iajret Iajret merged commit 906828e into master Feb 17, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-998 branch February 17, 2024 16:21
AnywayFarus added a commit that referenced this pull request Feb 17, 2024
Iajret pushed a commit that referenced this pull request Apr 16, 2024
* Small optimization for Aura Healing (#82674)

## About The Pull Request

- Makes use of the byond internal optimization for `view` to speed up
aura healing a tiny bit.
- In case you didn't know, byond has an optimization for `view` which
speeds up iterating over objects in view if you provide it a type. This
use of a ternary (likely) prevented this optimization from kicking in,
and since worst-case we're doing view(7) it can add up.
- Test case: 2 staff of Ascelpius users surrounded by 15 humans and 15
random objects constant being damaged. ~8 minutes of testing.

   - Profile:
```
/datum/component/aura_healing/proc/heal_old   0.789        6.590        6.596        0.000         4076
/datum/component/aura_healing/proc/heal_new   0.682        6.443        6.442        0.000         4081
```

- Generalizes `SSaura_healing` to `SSaura`, makes "damage aura"
component (which is totally 99% copied from "healing aura" but that's
for another pr) use it as well

## Changelog

:cl: Melbert
refactor: Staff of Healing should perform slightly better. 
/:cl:

* Small optimization for Aura Healing

---------

Co-authored-by: MrMelbert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants