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] Removes the old toolbox arena code #2179

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1191
Original PR: tgstation/tgstation#81707

About The Pull Request

This machine was added 5 years ago to be used for events like the toolbox tournament, but it has since been deprecated and unused (toolbox tournament has made their own version of the console that works much better and uses TGUI), making this machine not worth touching.
After some discussions in discord, we said the better option would be to remove this machine and leave the toolbox tournament branch keep using their machine, rather than to port the toolbox version to TG where it will continue to not be used in-game.

fun fact: This used to have a feature where it gave antag hud icons to the members of each team, it was removed when antag huds got refactored 3 years ago, hence why I'm removing an antag hud icon.

Why It's Good For The Game

Already explained, this machine has no purpose as it is now, and a new tgui UI won't save it.

Changelog

Nothing player-facing.

* Removes the old toolbox arena code (#81707)

## About The Pull Request

This machine was added 5 years ago to be used for events like the
toolbox tournament, but it has since been deprecated and unused (toolbox
tournament has made their own version of the console that works much
better and uses TGUI), making this machine not worth touching.
After some discussions in discord, we said the better option would be to
remove this machine and leave the toolbox tournament branch keep using
their machine, rather than to port the toolbox version to TG where it
will continue to not be used in-game.

fun fact: This used to have a feature where it gave antag hud icons to
the members of each team, it was removed when antag huds got refactored
3 years ago, hence why I'm removing an antag hud icon.

## Why It's Good For The Game

Already explained, this machine has no purpose as it is now, and a new
tgui UI won't save it.

## Changelog

Nothing player-facing.

* Removes the old toolbox arena code

---------

Co-authored-by: John Willard <[email protected]>
@Iajret Iajret merged commit 324ee09 into master Feb 29, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1191 branch February 29, 2024 18:39
Iajret pushed a commit that referenced this pull request Apr 25, 2024
* ChemMaster 3000 printing speed based on servo tier (#82836)

## About The Pull Request

Reverts the maximum number of containers (pills/patches/etc.) the
ChemMaster 3000 can print back to 50. (Recently changed to 12.5 per tier
in tgstation/tgstation#82002) Instead, the servo
tier determines the speed the batch is printed at.


![image](https://github.com/tgstation/tgstation/assets/83487515/d58a2325-55ff-4087-b73b-9d08ebdf98e5)


![image](https://github.com/tgstation/tgstation/assets/83487515/5d078957-dbc4-483f-8e38-62a815370f66)

## Why It's Good For The Game

Refilling the buffer and pressing print for every 13 pills is pointless
busywork when there is already a delay now implemented per pill printed.

## Changelog

:cl: LT3
balance: ChemMaster can again print a maximum of 50 pills/patches per
run
balance: Higher tier servos increase ChemMaster printing speed
/:cl:

* ChemMaster 3000 printing speed based on servo tier

---------

Co-authored-by: lessthanthree <[email protected]>
ReezeBL pushed a commit that referenced this pull request Apr 26, 2024
…3056)

* ChemMaster 3000 printing speed based on servo tier (#82836)

## About The Pull Request

Reverts the maximum number of containers (pills/patches/etc.) the
ChemMaster 3000 can print back to 50. (Recently changed to 12.5 per tier
in tgstation/tgstation#82002) Instead, the servo
tier determines the speed the batch is printed at.


![image](https://github.com/tgstation/tgstation/assets/83487515/d58a2325-55ff-4087-b73b-9d08ebdf98e5)


![image](https://github.com/tgstation/tgstation/assets/83487515/5d078957-dbc4-483f-8e38-62a815370f66)

## Why It's Good For The Game

Refilling the buffer and pressing print for every 13 pills is pointless
busywork when there is already a delay now implemented per pill printed.

## Changelog

:cl: LT3
balance: ChemMaster can again print a maximum of 50 pills/patches per
run
balance: Higher tier servos increase ChemMaster printing speed
/:cl:

* ChemMaster 3000 printing speed based on servo tier

---------

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