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

[TG Mirror] Birdshot ordnance QoL #137

Merged
merged 1 commit into from
Oct 19, 2023
Merged

[TG Mirror] Birdshot ordnance QoL #137

merged 1 commit into from
Oct 19, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#24418
Original PR: tgstation/tgstation#79056

About The Pull Request

Fixes #79003

I've added a couple of useful tools to Birdshot's ordnance lab - a second RPD, and two holofan projectors.

image

I also removed one (1) cable from under the reinforced window by ordnance's front door. I didn't touch the other electrified windows in the science wing.

Why It's Good For The Game

The extra RPD lets two people comfortably work in ordnance, like they can on most maps, and the holofan projectors allow changes to be made to the burn or freeze chambers without disaster. Most other maps have these things available, so it seems reasonable to add them here.

As pointed out in #79003, a quick toxins moment is all it takes for the modified window to explode, and for the pressure to slam an unfortunate spaceman into an electrified grille until they die. This feels excessive to me, and the other ordnance windows aren't electrified - so I took the cable out.

Changelog

🆑 lizardqueenlexi
qol: Birdshot ordnance is now equipped with a second RPD and two holofan projectors.
qol: Ordnance mishaps on Birdshot are significantly less likely to slam you into an electrified window until you die.
/:cl:

* Birdshot ordnance QoL (#79056)

## About The Pull Request

Fixes #79003

I've added a couple of useful tools to Birdshot's ordnance lab - a
second RPD, and two holofan projectors.

![image](https://github.com/tgstation/tgstation/assets/105025397/8024799a-39e5-461f-9450-912a758f9673)

I also removed one (1) cable from under the reinforced window by
ordnance's front door. I didn't touch the other electrified windows in
the science wing.
## Why It's Good For The Game

The extra RPD lets two people comfortably work in ordnance, like they
can on most maps, and the holofan projectors allow changes to be made to
the burn or freeze chambers without disaster. Most other maps have these
things available, so it seems reasonable to add them here.

As pointed out in #79003, a quick toxins moment is all it takes for the
modified window to explode, and for the pressure to slam an unfortunate
spaceman into an electrified grille until they die. This feels excessive
to me, and the other ordnance windows aren't electrified - so I took the
cable out.
## Changelog
:cl:
qol: Birdshot ordnance is now equipped with a second RPD and two holofan
projectors.
qol: Ordnance mishaps on Birdshot are significantly less likely to slam
you into an electrified window until you die.
/:cl:

* Birdshot ordnance QoL

---------

Co-authored-by: lizardqueenlexi <[email protected]>
@Iajret Iajret merged commit e31e558 into master Oct 19, 2023
23 checks passed
@Iajret Iajret deleted the upstream-mirror-24418 branch October 19, 2023 08:28
Steals-The-PRs added a commit that referenced this pull request Oct 19, 2023
Iajret pushed a commit that referenced this pull request Dec 28, 2023
* Makes immerse use weakrefs (#80594)

## About The Pull Request
Immerse was causing harddels due to it having references to mobs. Makes
it use weakrefs for mobs instead.

## Why It's Good For The Game
Immerse would cause harddels if a mob was deleted while it was in it's
list. It could probably also happen if a turf was deleted too, but doing
that here would be much harder.

no CL since nothing playerfacing

* Makes immerse use weakrefs

---------

Co-authored-by: Arturlang <[email protected]>
Co-authored-by: NovaBot <[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