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] cell chargers now bypass APCs #2749

Merged
merged 1 commit into from
Apr 5, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1815
Original PR: tgstation/tgstation#82309

About The Pull Request

This makes cell chargers and suit storage units draw from the grid before the local apc

Why It's Good For The Game

Upgraded Cell chargers have a charging power of 1MW leading to them instantly draining the apc of any room they are in,
this Pr makes them draw from the grid preventing immediate blackout.
This is a stopgap until someone smarter than me changes power values so a pocket-sized battery won't require the same power to charge then over 600 average suburban homes.

Changelog

🆑 Blacklist897
balance: suit and cell chargers should draw from grid preventing instant apc blackouts in most cases.
/:cl:

* cell chargers now bypass APCs (#82309)

## About The Pull Request
This makes cell chargers and suit storage units draw from the grid
before the local apc

## Why It's Good For The Game
Upgraded Cell chargers have a charging power of 1MW leading to them
instantly draining the apc of any room they are in,
this Pr makes them draw from the grid preventing immediate blackout. 
This is a stopgap until someone smarter than me changes power values so
a pocket-sized battery won't require the same power to charge then over
600 average suburban homes.

## Changelog
:cl:
balance: suit and cell chargers should draw from grid preventing instant
apc blackouts in most cases.
/:cl:

* cell chargers now bypass APCs

---------

Co-authored-by: Blacklist897 <[email protected]>
@Iajret Iajret merged commit fd13880 into master Apr 5, 2024
26 checks passed
@Iajret Iajret deleted the upstream-mirror-1815 branch April 5, 2024 13:29
AnywayFarus added a commit that referenced this pull request Apr 5, 2024
Iajret pushed a commit that referenced this pull request May 30, 2024
* Replaces `move_on_shuttle` with a trait (#83550)

## About The Pull Request

`move_on_shuttle` was a variable that existed on `/mob` despite only
ever being used by cameras. This PR downgrades the variable to only
exist on `/mob/camera` types, but have the only thing that variable do
is add a trait that blocks movement on shuttles if not allowed. This
allows us to check the variable without casting to camera, which is what
some code was un-necessarily doing. This also retainst the ability to
add the trait later on to different mobs who we may not want to be on
shuttles that aren't `/mob/camera`.
## Why It's Good For The Game

There's no point for this variable to live on `/mob` if all it can do is
clog up the VV screen, especially if the current utilization for it is
only cameras. Let's move it off `/mob` and turn it into a trait so it
still has the similar utilization on a global level without typecasting
- as well as make it actually work since needless typecasting was
introducing a minor bug
## Changelog
Irrelevant.

---------

Co-authored-by: Time-Green <[email protected]>

* Replaces `move_on_shuttle` with a trait

---------

Co-authored-by: san7890 <[email protected]>
Co-authored-by: Time-Green <[email protected]>
Co-authored-by: NovaBot13 <[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