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

HoS, warden and officers no longer spawn with a stunbaton in their bags. #62861

Merged
merged 2 commits into from Nov 17, 2021

Conversation

Ghommie
Copy link
Member

@Ghommie Ghommie commented Nov 16, 2021

About The Pull Request

The HoS, warden and officers no longer spawn with a stunbaton in their bags.

Why It's Good For The Game

More often than not, we end up with every security role (minus det) having two stunbatons since both their outfits and their closets have a stunbaton. This entails situations in which power cell drainage or losing the first stunbaton in a way that doesn't result in them becoming the receiving end of the stick would hardly impact their gameplay because of the backup one in their bag. It's also worth mentioning stunbatons are pretty powerful (the nerf they received 2 years ago helped, but the time window between the first and second hit is damn short) and most of their supposed counters (like stims, maint pump-up) barely help against the stamina loss and the knockdown right now.
In short, getting in a melee scuffle an officer can be tough, disposing his baton only for him to whip out another is just frustrating.
This also means they won't be as battle-ready but the chances are they'll take a walk to the brig and gear up anyway. It also adds a scarcity factor. Any extra security baton taken is a baton deprived from new officers, so that security and his officers have a reason to order stunbatons from cargo and interact with one another, but i'm just speculating here.
Here's Orange's blessing:
citrus hath spoken

Changelog

🆑
balance: The HoS, warden and officers no longer spawn with a stunbaton in their bags.
/:cl:

@tgstation-server tgstation-server added the Balance Changes to functionality that modifies how effective certain methods are at powergaming label Nov 16, 2021
@JohnFulpWillard
Copy link
Member

the massive hos buff was uncalled for but generally this really feels needed. Is cargo still the only way to get new batons?

@Ghommie
Copy link
Member Author

Ghommie commented Nov 16, 2021

the massive hos buff was uncalled

Is it? Talking from experience, telebatons are kinda slow.

Is cargo still the only way to get new batons?

Well, you can craft stunprods.

@ATH1909
Copy link
Contributor

ATH1909 commented Nov 16, 2021

who tf actually takes/keeps both batons

@ATH1909
Copy link
Contributor

ATH1909 commented Nov 16, 2021

and most of their supposed counters (like stims, maint pump-up) barely help against the stamina loss and the knockdown right now.

gee it's almost like we should be buffing pump-up

which I've tried to do before, btw

@optimumtact
Copy link
Member

there are not meant to be counters to stunbatons, other than staying out of melee distance.

@RegJackson
Copy link

Stunbatons don't spawn in the armory. Your PR and your blessing implies they do. This is wrong.

@anconfuzedrock
Copy link
Contributor

Note that there are not, in fact, batons in the armory. or maybe there are on like one map so I'm going to have people insulting me.

@Ghommie Ghommie changed the title Warden and officers no longer spawn with a stunbaton in their bag. HoS has a telebaton instead. HoS, warden and officers no longer spawn with a stunbaton in their bags. Nov 16, 2021
@RegJackson
Copy link

RegJackson commented Nov 16, 2021

The stunbaton you spawn with has always seemed more to me as a tool to prevent getting spawncamped during revs and as a fallback when all of sec has been bombed after a late arrival (A situation that I have actually been in and only won because of my starting equipment.)

I, personally, think a better fix would be to simply remove the baton from the belts in lockers. This in itself nerfs the usual locker break-ins that happen with arrivals, but personally I don't think that's too bad. That's just me though.

@iprice
Copy link
Contributor

iprice commented Nov 16, 2021

Inclined to agree, remove the station spawns and keep it as a player spawn if you want to limit. Otherwise people will loot multiple at shift start in a race (like insuls) and then you have seccies without a baton. Particularly late joins. Concerns about how seccies without the staple non lethal tool end up interacting, and while cargo can order more, getting them to actually do so is a whole other kettle of fish (have very poor experiences of inter departmental dependencies with atmos/cargo/science as CMO main)

@BlueMemesauce
Copy link
Contributor

I think it makes more sense to have the stun removed from the lockers and instead spawn in bags.
Otherwise sec could just loot the lockers and take two stun batons from the belts.

@BlueMemesauce
Copy link
Contributor

who tf actually takes/keeps both batons

Everyone?

@optimumtact
Copy link
Member

optimumtact commented Nov 16, 2021

I'm very much okay with the impact of secs having to deal with some limits on their stunbatons, and that inviting some decision making.

I'm also okay with rev's having an upperhand on late join sec who join after sec is blown up.

Neither strike me as completely unbalancing for security and present only minor inconveniences, that will require some active player decisions.

@cacogen
Copy link
Contributor

cacogen commented Nov 16, 2021

there are not meant to be counters to stunbatons, other than staying out of melee distance.

It's clear that you don't know what you're talking about with this. https://github.com/tgstation/tgstation/blob/master/code/modules/reagents/reagent_containers/hypospray.dm#L293

@optimumtact
Copy link
Member

it may shock you but i'm aware of pump up, and it's heavily nerfed from it's previous incarnations for a reason cacogen.

@BlueMemesauce
Copy link
Contributor

It doesnt actually impact Security as much as it seems. Because of the new update, now security has a cargo console that they can use to order free stun batons.

@anconfuzedrock
Copy link
Contributor

It doesnt actually impact Security as much as it seems. Because of the new update, now security has a cargo console that they can use to order free stun batons.

they're not free, taking those batons means you won't be able to buy anything else, and most options are better than batons.

@optimumtact
Copy link
Member

oh no you have to make a tradeoff? that's gonna hurt the gameplay

Copy link
Member

@Mothblocks Mothblocks left a comment

Choose a reason for hiding this comment

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

I always thought this was weird when a sec-belt is basically always needed anyway

@Ghommie
Copy link
Member Author

Ghommie commented Nov 16, 2021

Stunbatons don't spawn in the armory. Your PR and your blessing implies they do. This is wrong.

Someone has also told me a few maps have up to 11 or so closets with batons in them. I may work on lowerin the number to 6 or 7 (whichever is the number of maximum officer slots, unless more are opened by the HoP) and add one or two to the armory if given the chance later.

@CRITAWAKETS
Copy link
Contributor

sec still spawns with a disabler and armor when the armory has disablers and the closets have armor.

you know what to do

@wesoda25
Copy link
Contributor

Could be interesting

@GuillaumePrata
Copy link
Contributor

We already have officers that loot multiple Flashbangs (and in the past, flashes) from all belts in the lockers, I assume the same will happen with the baton and this won't fix the issue of backup baton, just make things annoying if you have some selfish officers around.
The way to stop backup batons is the same way the bag of guns was fixed, making them bulky, that will make things quite hard for traitors that like to loot them tho.
Regardless of both, scarcity alone is good so this is a nice step.

@Draggeru
Copy link
Contributor

Antags already have a massive advantage in a one-on-one situation, I see this fucking over late join officers. Who might I add have no idea what's going on in the round until they actually join. I would be more okay with this pr if it took the stun batons and put them in the armory so they could still be claimed but a straight removal makes me fear for balance and potentially removing sec mains desire to play security if they get screwed over too often because they don't have a melee counter. Disablers don't really do shit especially if your aim is bad.

@ExcessiveUseOfCobblestone
Copy link
Contributor

Skill issue but actually

@ATH1909
Copy link
Contributor

ATH1909 commented Nov 16, 2021

Antags already have a massive advantage in a one-on-one situation, I see this fucking over late join officers. Who might I add have no idea what's going on in the round until they actually join. I would be more okay with this pr if it took the stun batons and put them in the armory so they could still be claimed but a straight removal makes me fear for balance and potentially removing sec mains desire to play security if they get screwed over too often because they don't have a melee counter. Disablers don't really do shit especially if your aim is bad.

maybe we could make latejoin sec officers start with batons, but not roundstart officers?

@Lauwma
Copy link

Lauwma commented Nov 16, 2021

Just the fact that you lied in order to get maintainer approval should get this PR closed
The armory does not start with batons, the only batons are the ones in the closets.

Removing batons from bags makes so late joins officers won't have anything because of every heads of staff tiding for belt + huds glasses

You should probably play the game before making PRs to be honest

@SinfulBliss
Copy link

the massive hos buff was uncalled

Is it? Talking from experience, telebatons are kinda slow.

HoS doesn't get telebaton shiftstart, he gets stunbaton in bag like officers. Only difference shiftstart is he comes with sechuds and an egun instead of a disabler.

@SuperNovaa41
Copy link
Contributor

the massive hos buff was uncalled

Is it? Talking from experience, telebatons are kinda slow.

HoS doesn't get telebaton shiftstart, he gets stunbaton in bag like officers. Only difference shiftstart is he comes with sechuds and an egun instead of a disabler.

this pr originally gave HOS telebaton

@Ghommie
Copy link
Member Author

Ghommie commented Nov 16, 2021

Another alternative would be security still spawn with their batons but :

* Stun baton become bulky.

* Only fit on suit slot/belt slot.

* Security belt get a energy bola instead.

For that scenario, I'd recommend making the security belts bulky but still able to fit one stunbaton (the storage component has a ton of variables but i'm not sure there's one that puts a cap on how many items of specific types can be inserted) to avoid diminishing their usefullness or increasing the tedium of inventory management more than necessary.

@SuperNovaa41
Copy link
Contributor

Another alternative would be security still spawn with their batons but :

* Stun baton become bulky.

* Only fit on suit slot/belt slot.

* Security belt get a energy bola instead.

or, they could just get a sec belt, and get their baton that way, like they always do???

@castawaynont
Copy link
Contributor

Give secoffs belts shift start then, remove em from closets

@SinfulBliss
Copy link

SinfulBliss commented Nov 16, 2021

Another alternative would be security still spawn with their batons but :

* Stun baton become bulky.

* Only fit on suit slot/belt slot.

* Security belt get a energy bola instead.

For that scenario, I'd recommend making the security belts bulky but still able to fit one stunbaton (the storage component has a ton of variables but i'm not sure there's one that puts a cap on how many items of specific types can be inserted) to avoid diminishing their usefullness or increasing the tedium of inventory management more than necessary.

Ghommie, brother, pal. What sec will simply do is carry the second baton in bag so they can toggle it out as if it were the first baton. It's really just added tedium that sec will have to work around to get a setup that's practical for the round.

If only suit/belt, simply put disabler in bag and baton in belt and suit storage. Energy bola shiftstart is pointless since they can be bought for 7 credits from a vendor.

@GuillaumePrata
Copy link
Contributor

Ghommie, brother, pal. What sec will simply do is carry the second baton in bag so they can toggle it out as if it were the first baton. It's really just added tedium that sec will have to work around to get a setup that's practical for the round.

If only suit/belt, simply put disabler in bag and baton in belt and suit storage. Energy bola shiftstart is pointless since they can be bought for 7 credits from a vendor.

You're ignoring the massive buff it would be for sec if randoms can't just shove you and bag your baton.
I would gladly trade the tedium or loss of multi baton meta if we get traitors and randoms having to make a choice between trading their belt/exo slot for a visible baton or not instead of just having one in the bag whenever it is possible.

@TheBonded
Copy link
Contributor

I play security often enough and I rarely if ever see the extra baton come into use. Sec officers nearly always just go and grab a belt, and either leave the extra baton in security or forget about it in their bag.

In fact, the ONLY time I've ever seen a security officer use the second baton was when they rigged the battery on it with plasma and would throw it at criminals, but intentionally miss, so when the criminal picked it up and tried to stun the officer they would explode and die, with minimal harm to the officer.

Besides, if an officer wants a second baton, what's stopping them from just grabbing another from one of the other lockers? There's almost always a few extra belts, so this change really wouldn't nerf sec all that much, if at all.

@ExcessiveUseOfCobblestone
Copy link
Contributor

How does one know the frequency at which people use the second baton if they are both pulled from their butt, can be switched out seamlessly, and look like exact copies?

If they aren’t using the baton that seems like there’s a failing where a baton with the normal battery can tackle even a group antagonist and not have to consider the amount of times they lock down a user

@adamkad1
Copy link

HURR DURR IM GONNA STEAL ALL THE BATONGS

@borb197
Copy link

borb197 commented Nov 16, 2021

There are no extra batons in the armoury, although there are security belts with one baton each in the lockers outside it. Honestly, just remove the batons inside those and let officers keep their roundstart batons, also cargo still remains a super easy way to get batons, all this PR does is grief sec offs / hos / warden roundstart or when they latejoin they're already vulnerable enough, let alone without a baton.

@timothymtorres
Copy link
Member

In fact, the ONLY time I've ever seen a security officer use the second baton was when they rigged the battery on it with plasma and would throw it at criminals, but intentionally miss, so when the criminal picked it up and tried to stun the officer they would explode and die, with minimal harm to the officer.

Besides, if an officer wants a second baton, what's stopping them from just grabbing another from one of the other lockers? There's almost always a few extra belts, so this change really wouldn't nerf sec all that much, if at all.

This is pretty robust.

Having an extra baton isn't that big of a deal. There are plenty of useful stuff you can replace it with.

@wesoda25
Copy link
Contributor

Having a (mostly) static supply of materials to manage - which all crewmembers can potentially get their hands on - is more interesting to me than just giving one to every member of security. The latter just seems to provide a lot less agency for the crew. Creating a scarcity is an interesting dynamic to me, especially considering how powerful batons are. Hopefully this encourages a different approach to security for some people, although as Lukas said the concern is that it could just be resorting to lethals sooner than later.

@cacogen
Copy link
Contributor

cacogen commented Nov 17, 2021

inb4 stun baton crate every single round

@TheBonded
Copy link
Contributor

inb4 stun baton crate every single round

@cacogen, the stun batons secoffs spawn with are rarely used as is, since they have sec belts in their lockers that contains a baton already in them. taking the time to buy a baton crate would be a huge waste since it would always be faster to walk into security and just grab a belt, which officers already do every round. this effectively changed nothing, except making departmental secoffs slightly more vulnerable (but still less vulnerable than normal crew) during the first 30 seconds of a round while they walk to sec to get a belt (which they already did before this change anyways).

@optimumtact
Copy link
Member

inb4 stun baton crate every single round

you say this like it's somehow a bad thing, it's not, it's actually better than cargo going unused.

@cacogen
Copy link
Contributor

cacogen commented Nov 17, 2021

@TheBonded

My understanding and correct me if I'm wrong is that if anybody grabs a baton out of one of these belts or the belt itself, then security is now down a baton and provided enough officers join someone will miss out. This means on highpop rounds it's likely somebody will not get a baton without having to spend the best part of ten minutes getting one from cargo. Once you have to do this over multiple rounds it adds up to a huge chunk of downtime mucking around waiting on somebody else to do their job.

you say this like it's somehow a bad thing, it's not, it's actually better than cargo going unused.

People would use cargo more if it were consistently staffed and faster. The shuttle takes too long and loading and unloading crates takes too long and is not enjoyable to do. Most people do not load crates or stamp manifests and cargo techs are often absent from their jobs. I myself enjoy cargo and always load and unload crates and stamp manifests but it's tedious as fuck and I'm usually the only one doing it.

As an aside, stamping manifests has been hugely disincentivised by a recent change that makes it penalise you I think 600 points for erroneously approving a manifest while correctly approving one only gives you 80. Almost nobody checks manifests and it would grind cargo to a standstill if they did. It's also difficult to check a manifest for a crate that somebody else has to open. It just leads to people not stamping them at all. In which case, why even have the mechanic? It was added as a reference to Papers, Please and I thought it was tedious to the point of being unrealistic to expect people to do it then and I feel similarly now.

@optimumtact
Copy link
Member

People would use cargo more if it were consistently staffed and faster. 

and it won't be staffed until it's relevant, don't try to tie me up in a chicken and egg problem, because i've already picked a side.

@LemonInTheDark
Copy link
Member

I have been in this place before, flying down the street

@NanoCats

This comment has been minimized.

@SinfulBliss
Copy link

SinfulBliss commented Nov 19, 2021

inb4 stun baton crate every single round

@cacogen, the stun batons secoffs spawn with are rarely used as is, since they have sec belts in their lockers that contains a baton already in them

Completely incorrect.. Every experienced security officer I know keeps two batons on them. It's so easy to lose one of your batons or have the charge run out on the other. But the real reason is because, in a baton-joust, you will likely drop your baton after being stunned, and being able to whip out your backup and get the second hit in to stamcrit will save your ass.

Likewise if you're shoved with a baton out, your baton drops, you can whip out the second and stun whoever is trying to loot it. This happens super often by the way.

Mothblocks pushed a commit to Mothblocks/tgstation that referenced this pull request Nov 22, 2021
Mothblocks pushed a commit to Mothblocks/tgstation that referenced this pull request Nov 22, 2021
Wallemations added a commit to Wallemations/heavenstation that referenced this pull request Nov 24, 2021
* Automatic changelog generation for PR tgstation#62531 [ci skip]

* Fixes cargo console breaking when you ordered canisters, and makes it break softer when it does anyways (tgstation#62829)

🆑
fix: cargo consoles aren't breaking when department orders buys a canister. seriously though who buys canisters man
/🆑

* Automatic changelog generation for PR tgstation#62829 [ci skip]

* Wings negate fall damage from short falls (remake) (tgstation#62800)

Co-authored-by: Mothblocks <35135081+Mothblocks@users.noreply.github.com>

* Automatic changelog generation for PR tgstation#62800 [ci skip]

* Gives Space Pirates a bonus to hijack speed on the shuttle. (tgstation#62806)

If a pirate pulls this off, they've got to be the best pirate I've ever seen.

* Automatic changelog generation for PR tgstation#62806 [ci skip]

* Fixes the foreach and filter circuit component (tgstation#62798)

Co-authored-by: Watermelon914 <3052169-Watermelon914@users.noreply.gitlab.com>

* Automatic changelog generation for PR tgstation#62798 [ci skip]

* Adds a worn on belt sprite for seclites (tgstation#62805)

* big light

* dah, verry good

* Automatic changelog generation for PR tgstation#62805 [ci skip]

* Stops HFRs (and other INDESTRUCTIBLE machinery) from being able to be blown up if an explosion's epicenter is on the tile (tgstation#62164)

* Stops HFRs from being able to be blown up if an explosion's epicenter is on the tile

* Stops stationloving objects from being able to be teleported to HFR core

* Automatically add PREVENT_CONTENTS_EXPLOSION_1 to all INDESTRUCTIBLE machinery

* Document reason for PREVENT_CONTENTS_EXPLOSION_1 flag being needed

* Automatic changelog generation for PR tgstation#62164 [ci skip]

* The Nuke kills everyone when it goes off. (tgstation#62807)

This does not make the nuke explode like it used to.
The nuke will now dust you in a similar way to the AI doomsday device when it goes off. This will only happen if the nuke successfully detonated on the station and not on a near miss. Otherwise, it kills everything on the z-level.

There has been code in the game for an extended period of time that was supposed to make this happen, but I believe it was bugged and did not properly give the z-level of the nuke in the explosion.

It should be noted that gameplay-wise this makes it so antagonists who need to escape will no longer get a free pass for being on the station when the nuke goes off, but they will still be counted as escaped if they are in lavaland/another z-level. I would say that this is only good for the game but I understand if it's controversial.

The original code gibbed people on the z-level, although I've considered changing it to dusting. I would like opinions on this or alternative methods of death.

fixes tgstation#62788

* Automatic changelog generation for PR tgstation#62807 [ci skip]

* Double beds can now have two people buckled to them at a time. Also doubles the material costs to make them and their sheets. (tgstation#62813)

* Automatic changelog generation for PR tgstation#62813 [ci skip]

* Hydroponics Tray USB Port (plus a tiny refactor needed to fit it in) (tgstation#62652)

Adds a USB port to constructable hydroponic trays. I had to refactor some hydro tray code, which is kinda old, to fit the USB circuit and the new signals in.

* Automatic changelog generation for PR tgstation#62652 [ci skip]

* Adds some basic sanity to the plumbing bottler (tgstation#62823)

* Adds some basic sanity to the plumbing bottler

* Update code/modules/plumbing/plumbers/bottler.dm

Co-authored-by: Emmanuel S. <emmanuelssr@gmail.com>

* Update code/modules/plumbing/plumbers/bottler.dm

Co-authored-by: Emmanuel S. <emmanuelssr@gmail.com>

Co-authored-by: Emmanuel S. <emmanuelssr@gmail.com>

* The atheists fedora now makes you try to get further from god (tgstation#62821)

Co-authored-by: Thunder12345 <stewart@critar.demon.co.uk>

* Automatic changelog generation for PR tgstation#62821 [ci skip]

* Automatic changelog compile [ci skip]

* Manually correct dir on every special airalarm (tgstation#62790)

* Manually correct dir on every special airalarm

Also fix the placement of the box mixing air alarm.

* Forgot to check in MetaStation

* Automatic changelog generation for PR tgstation#62790 [ci skip]

* Changed fermichem related values for Inacusiate and Multiver (tgstation#62467)

* Automatic changelog generation for PR tgstation#62467 [ci skip]

* Allows plates on trays (tgstation#62814)

* Automatic changelog generation for PR tgstation#62814 [ci skip]

* Smooths the gliding of moving trams and its passengers. (tgstation#62587)

This makes the glide size of movables inside a tram match that of the platform they are on when the tram is moving (which should in turn match the tramprocess substystem wait).

This will stop slower mobs from looking all jittery when moving on a tram at the cost of a blockier movement while moving inside a moving tram*. This is a small price to pay also considering there's not much space to move around inside the tram anyway and it isn't as annoying.

*Byond/our code doesn't support concurrent glide sizes ( basically if you try to change it midmovement, it'll also update the speed of the current movement step). There's no efficient and universal solution to this.

* Automatic changelog generation for PR tgstation#62587 [ci skip]

* carpswag (tgstation#62811)

expansion: Carpskin suits and fedoras can now be crafted using carp scales.

* Automatic changelog generation for PR tgstation#62811 [ci skip]

* New/Modification sprite for the barrier grenade (tgstation#62775)

imageadd: New barrier grenade sprite

* Automatic changelog generation for PR tgstation#62775 [ci skip]

* FindOccupationCandidates will no longer die if the client disappears halfway through the proc. (tgstation#62819)

Co-authored-by: Mothblocks <35135081+Mothblocks@users.noreply.github.com>

* Doubles slow asset send rate (tgstation#62845)

Newer byond clients now properly batch up `browse_rsc` sends so this clogging up the `browse()` queue is less of a concern.

* Comfy chair code improvements (tgstation#62842)

* Automatic changelog generation for PR tgstation#62842 [ci skip]

* Da plate update: New plate sizes and breakable plates (tgstation#62804)

About The Pull Request

This PR adds small and large plates and allows you to break plates by throwing them.

bild
bild
Why It's Good For The Game

Having larger plates allows you to more easily run the buffet gimmick as you can fit more items on the large plates.

Having large plates also allows for more space for creative food plating.

The small plates lets the chef serve small food items such individual cake slices without the plate looking mismatched.

Plate breaking improves immersion and allows unsavoury characters such as mimes and syndicates to stir up trouble.
Changelog

cl
expansion: Small and large plates have been added to the kitchen vend.
expansion: Plates will now break when thrown.
/cl

* Automatic changelog generation for PR tgstation#62804 [ci skip]

* UpdatePaths: Handle lists containing strings (tgstation#62595)

* UpdatePaths: Handle lists containing strings

The split_re regex considers two alternatives when splitting out property
values.
The first alternative, "(?:.+?)" looks for a value that starts and
ends with a double quote, containing any characters between, but using
the smallest length possible.
The second alternative, [^";]* looks for a value that does not contain
any double quotes or semicolons.

Neither of these cases will handle a value such as network = list("fsc");

To handle this case without a major rewrite, the second alternative is
updated to [^"][^;]+ which looks for a value that does not start with a
double quote and does not contain a semicolon.

* UpdatePaths Support @old;... too

This strips the trailing semicolon, if present, which it will be if
you're overriding an old value after specifying @old.

* UpdatePaths: Handle single char case

Whoops

* Automatic changelog generation for PR tgstation#62595 [ci skip]

* Adds New Aquarium Fish! (tgstation#62840)

* fish

* fixes broken stuff

* no emulsijack betrayal, emuslijacks will now reproduce

* Automatic changelog generation for PR tgstation#62840 [ci skip]

* Modern Armor For The Modern Age - Marine ERT's Armor Update (tgstation#62714)

* Automatic changelog generation for PR tgstation#62714 [ci skip]

* Wiremod assoc_literal circuit component was missing from the lathe (tgstation#62818)

Was looking for a way to make a cache list when I noticed the assoc_literal component wasn't in the lathe
I rather cache a list of stuff than continuously run foreach/searchs on data from the ntnet or otherwise.

* Automatic changelog generation for PR tgstation#62818 [ci skip]

* Reconvert all diagonal cameras, properly this time (tgstation#62789)

* Automatic changelog generation for PR tgstation#62789 [ci skip]

* Automatic changelog compile [ci skip]

* bad mafia (tgstation#62874)

* Automatic changelog generation for PR tgstation#62874 [ci skip]

* nitryl and stimulum merge in nitrium (tgstation#62061)

This PR is the first of a few were i'll be merging similar working gases into one.
This time is Nitryl and Stimulum. They'll be merged into Nitrium , a brown gas with both features of the two gases
The main scope is to add dept to atmos by removing bloated content and/or repeated content that has never seen the lights of the day (how many times have you seen both gases made and used at the same time?)
The PR so far:
-removed nitryl
-removed stimulum
-merged them into Nitrium (Nitrium is now made with trit, nitrogen and bz from a minimum temperature of 1500 K)
-made Nitrium have both gases features such as fast movements and sleep and stun immunity but increased damage taken
-Nitrium can make crystals that spread the chemicals with a cloud (is still far more efficient to just breathe the gas)

Less rare gases, going towards a better atmos gameplay loop

* Automatic changelog generation for PR tgstation#62061 [ci skip]

* HoS, warden and officers no longer spawn with a stunbaton in their bags. (tgstation#62861)

* Automatic changelog generation for PR tgstation#62861 [ci skip]

* Fixes the wrong cinematic playing on nukeop win (tgstation#62877)

* Automatic changelog generation for PR tgstation#62877 [ci skip]

* Gets Rid Of An Errant Space From The Baton Knockdown Message (tgstation#62865)

* Fixing quantum pads teleporting themselves, or How I Stopped People Stealing The Self-Destruction Terminal (tgstation#62849)

* Automatic changelog generation for PR tgstation#62849 [ci skip]

* Splits antag huds into their own .dmi file and de-hardcodes antag huds (tgstation#62767)

* clown and mime can access service orders (tgstation#62882)

* Automatic changelog generation for PR tgstation#62882 [ci skip]

* Fixing floorbot (tgstation#62878)

Not tested, but YOLOmerge requested

* Automatic changelog generation for PR tgstation#62878 [ci skip]

* Fixes some issues with the PDA UI (tgstation#62864)

* Automatic changelog generation for PR tgstation#62864 [ci skip]

* Admin: Fix Camera Report verb (tgstation#62866)

There is still the potential for false positives since the way diagonal
cameras were made didn't exactly line up with where their direction was.

However, this brings functionality back to parity with before the
dir-sanity PR.

* Automatic changelog generation for PR tgstation#62866 [ci skip]

* Automatic changelog compile [ci skip]

* Makes the "Ian's Adventure" station trait more interesting by making the dog deadchat controllable plus extra lives. (tgstation#62617)

About The Pull Request

Makes the "Ian Adventure" station trait more interesting by giving him deadchat control (democracy mode, 3 seconds cooldown on inputs) and a couple extra lives* to survive early round tiding.

*Basically, a new component that respawns the critter when he dies. The component itself is simple enough, but it sends a signal each respawn to allow other datums to expand this behavior how they want. I've contemplated adding a signal that can stop death, dusting and gibbing instead of respawning the mob at first but because death(), dust() and gib() were made with the assertion that the mob is always going to die I've quickly realized it'd require a refactor that's way too big and out of scope.

The deadchat control of corgis only include commands to change and drop hats, "speak" (random lines from the speak list of the mob) and spin (other than cardinal movement) for now. I'd have loved to add more complex commands (like, argumented) that can potentially be relayed to the AI controller, but I guess that'll have to wait until the thing gets refactored a little.
Why It's Good For The Game

"Ian's Adventure" is a really bland station trait at the moment. All it does is move Ian somewhere else at the start of the round. Even by the station traits standard of being small things this is insignificant, and also pretty bad considering Ian is not actually going on an adventure since he lacks the initiative to do anything being an npc dog.
This PR aims to breath fresh air into this station trait and push it toward a slighty more engaging direction, though it relies on observers to work, while still being a small """"cute"""" station trait.
Changelog

cl
expansion: The "Ian's Adventure" station trait now makes Ian deadchat controllable and gives him a couple extra lives (to survive early round tiding)
/cl

* Automatic changelog generation for PR tgstation#62617 [ci skip]

* Fixes loading funny admin nicknames   (tgstation#62881)

* Automatic changelog generation for PR tgstation#62881 [ci skip]

* implements pews needing a tool to be rotated (tgstation#62373)

About The Pull Request

    PR'ed for Hacktoberfest (so if this passes muster I'd appreciate it being tagged with hacktoberfest-accepted)
    Pews now require a wrench in hand to be rotated
    closes 

    Can rotate pews tgstation#50136
    image

Why It's Good For The Game

This prevents people from easily messing up the chapel and publicly placed pews
Changelog

cl
add: Wooden pews now require a wrench in hand to be rotated
/cl

* Automatic changelog generation for PR tgstation#62373 [ci skip]

* Fixes a few cargo exports problems. (tgstation#62686)

About The Pull Request

Cargo exports will now start deleting the thing and its contents only after everything has been sold and hopefully stop objects from being deleted before getting sold (no export datum actually deletes anything on sell_object()). This PR also removes a variable only used in one place (an admin only item created by ExcessiveUseOfCobblestone, who probably didn't know there's an argument that stops unsold items from getting deleted) and that may be source of harddels.
Why It's Good For The Game

This will fix tgstation#62644, perhaps some harddels and other oddities.
Changelog

cl
fix: Cargo exports will now start deleting items only after everything is sold. This will fix issues such as unachievable exports (like machine ones) and getting less credits than what export scanners says.
/cl

* Automatic changelog generation for PR tgstation#62686 [ci skip]

* Add randomized tracking beacons to all maps (tgstation#61673)

This takes the new random tracking beacon spawner and adds them to all maps. The only guaranteed tracking beacon that spawns is the one at arrivals. The rest have a 35% chance to spawn and are located in the following areas:

    teleporter
    AI Sat teleporter
    escape
    showcase
    bridge
    courtroom
    bomb testing range
    atmos
    xenobio secure chamber
    medbay
    engineering
    bar/cafe
    EVA
    virology
    dorms
    mech bay
    lavaland
    cargo
    shooting range/security
    chapel
    disposals
    hydro
    garden
    library
    gateway
    incinerator room
    vacant office

More roguelike randomization for every round is good for the soul.

* Automatic changelog generation for PR tgstation#61673 [ci skip]

* Fixes loading scars file (tgstation#62928)

a317825 broke loading default string files with load_strings_file(), since there was no default directory

* the ultimate hulk nerf (tgstation#62891)

* Automatic changelog generation for PR tgstation#62891 [ci skip]

* Micro optimizations: Now much less exhaustive (tgstation#62844)

* Fixes an oversight with tail pulling and the Bad Touch trait (tgstation#62784)

* Automatic changelog generation for PR tgstation#62784 [ci skip]

* Improve the bureaucracy system (tgstation#61323)

Adds a blank printing system.

Virtually every Space Station 13 server has its own system of bureaucracy. As a rule, it is some kind of Wiki page with markup tags and a few sample forms. And that's usually the end of it. With stamps and a filing system in place, people rarely use them. This PR takes it upon itself to remedy this situation by introducing form standards into the game itself and adding the ability to print them.

Co-authored-by: twilightwanderer <twilightwanderer@users.noreply.github.com>
Co-authored-by: tralezab <40974010+tralezab@users.noreply.github.com>
Co-authored-by: Aleksej Komarov <stylemistake@gmail.com>

* Automatic changelog generation for PR tgstation#61323 [ci skip]

* Proper custom job name support (tgstation#62803)

* Makes PDA crew manifest and crew monitoring computer respect trims for ordering and sorting

* Fix static typecheck issue

* Added initial rank tracking for IC command

* Automatic changelog generation for PR tgstation#62803 [ci skip]

* cyborgs doing research is now logged IC (tgstation#62898)

AI's doing research is logged to the RD, but Cyborgs doing it just shows up as 'Unknown', which makes no sense because some AI can order a Cyborg to do research on their behalf so the RD doesn't know who did it.
It also makes the RD's job easier to find out what rogue cyborg is doing research when told not to.

* Automatic changelog generation for PR tgstation#62898 [ci skip]

* Pacifists can no longer attach c4 to other living creatures. (tgstation#62903)

* Automatic changelog generation for PR tgstation#62903 [ci skip]

* Adds a new poster to commemorate the end of an era (tgstation#62885)

* Added a new poster

* Adds an extra layer of grime.

Co-authored-by: Farquaar <(tgGithubPW0)>

* Automatic changelog generation for PR tgstation#62885 [ci skip]

* Automatic changelog compile [ci skip]

* Add cosmetic random spawners to tramstation (tgstation#62753)

* Automatic changelog generation for PR tgstation#62753 [ci skip]

* Add cosmetic random spawners to Metastation (tgstation#62929)

* Add cosmetic random spawners to Meta

* Automatic changelog generation for PR tgstation#62929 [ci skip]

* Move conveyor belts to their own system, rename conveyor2 (tgstation#62911)

Renames conveyor2.dm to conveyor.dm, and removes the historical comment about "new conveyor belts". As a reminder, comments should be about what the code is now, not what it used to be. (Muh history -Lemon)

Moves conveyor belts to their own SS with an identical wait. It is very common, as an admin, for me to want to adjust conveyor belt processing, either to make it faster (for laughs) or to disable it when it is causing heavy lag. It is very difficult to do this (especially the former) because it just uses fast process.

* Icebox service department fixes (tgstation#62846)

* fixes to icebox service department

* stool dirs

* ducting

* Automatic changelog generation for PR tgstation#62846 [ci skip]

* Delta Ice Kilo Minor Map Fixes (tgstation#62848)

Deltastation Teleporter Shutters fixed

Icebox Service exit properly marked

Kilo Kitchen shutters fixed

* Automatic changelog generation for PR tgstation#62848 [ci skip]

* remove abandoned helper (tgstation#62886)

* Automatic changelog generation for PR tgstation#62886 [ci skip]

* Donuts found in sec vendors and boxes can now be used to make glazed donuts (tgstation#62910)

* Automatic changelog generation for PR tgstation#62910 [ci skip]

* Replacing more C-style for loops with the faster, traditional ones. (tgstation#62908)

* Fixes unpowered icebox mining doors (tgstation#62847)

* Automatic changelog generation for PR tgstation#62847 [ci skip]

* Evidence bag in the bag (tgstation#62913)

* Automatic changelog generation for PR tgstation#62913 [ci skip]

* prevents mindswaping into ghost poly (tgstation#62554)

About The Pull Request

Closes tgstation#62465

Also makes ghost poly technically designated as a boss in the code as their sentience type, as far as I can tell this doesn't actually change anything other than the ability to make it sentient, but it also makes sense for a creature effectively incapable of death to be designated as a boss in the code.
Why It's Good For The Game

As funny as it is, mindswapping (blood into a rainbow slime extract gives you a mindswap potion) into ghost poly, or otherwise making ghost poly sentient, is an exploit that should be patched. Ghost poly is as fast as a ghost, can move through walls, has godmode, and if you give it a fugu gland it can beat people up. It can also pick up things, such as the nuke disk, which would make nukeops rounds pretty unfair and unfun.
Changelog

cl
fix: Aspiring xenobiologists and ambitious wizards can no longer mindswap into ghost poly or otherwise make them sentient in an attempt to gain godmode + ghost speed + the ability to go through walls.
/cl

* Automatic changelog generation for PR tgstation#62554 [ci skip]

* Irish car bomb clown car interaction (tgstation#62889)

* Automatic changelog generation for PR tgstation#62889 [ci skip]

* Fixes stale data when a user performs a refresh of a TGUI page (tgstation#62930)

We should send a full update when a user refreshes a TGUI web page. Refreshes can be performed by pressing F5 while focused within a TGUI window.

* Automatic changelog generation for PR tgstation#62930 [ci skip]

* Fixes emptied shot glasses having the drinking glass filling icon. (tgstation#62920)

Made both drinking and shot glasses use the generic reagent_containers/update_overlay() method. Standardized drinking and shot glasses update_icon_state().

* Automatic changelog generation for PR tgstation#62920 [ci skip]

* Allows shells that requiring anchoring to take power from APCs (tgstation#62907)

Co-authored-by: Watermelon914 <3052169-Watermelon914@users.noreply.gitlab.com>

* Automatic changelog generation for PR tgstation#62907 [ci skip]

* Hotfix: Vendor, Crafting, Mulebots (tgstation#62863)

* Automatic changelog generation for PR tgstation#62863 [ci skip]

* Automatic changelog compile [ci skip]

* Exempts non-deadminned admins from all the chat filters so they can break the server rules and get banned if they want to. (tgstation#62944)

Co-authored-by: Mothblocks <35135081+Mothblocks@users.noreply.github.com>

* Automatic changelog generation for PR tgstation#62944 [ci skip]

* Allows players to send more visible adminhelps when no admins are on, which'll definitely alert admins (tgstation#62711)

Co-authored-by: Watermelon914 <3052169-Watermelon914@users.noreply.gitlab.com>

* Automatic changelog generation for PR tgstation#62711 [ci skip]

* Revamps the Space Hotel Ruin (tgstation#62659)

* Automatic changelog generation for PR tgstation#62659 [ci skip]

* Fixed Icebox Air Alarm (tgstation#62927)

* Automatic changelog generation for PR tgstation#62927 [ci skip]

* [MDB IGNORE] Repath wall reagent dispensers (tgstation#62593)

* Repath wall reagent dispensers

This changes /obj/structure/reagent_dispensers/{virusfood,peppertank}
to be subpaths of /obj/structure/reagent_dispensers/wall.

The WALLITEMS test now checks for all wall mounted reagent dispensers,
instead of just peppertank specifically.

> git ls-files '*.dmm' | xargs -d\\n sed -i 's@/obj/structure/reagent_dispensers/peppertank@/obj/structure/reagent_dispensers/wall/peppertank@g'
> git ls-files '*.dmm' | xargs -d\\n sed -i 's@/obj/structure/reagent_dispensers/virusfood@/obj/structure/reagent_dispensers/wall/virusfood@g'

* Automatic changelog generation for PR tgstation#62593 [ci skip]

* creativecommitname (tgstation#62935)

Co-authored-by: unknown <burgercoding@gmail.com>

* Automatic changelog generation for PR tgstation#62935 [ci skip]

* Automatic changelog compile [ci skip]

* Votes work again (tgstation#62973)

So, it was runtiming because it wasn't checking for the length of the list, so it was failing to display the results and failing to do anything about the results of said vote. That means map votes didn't do anything.

* Automatic changelog generation for PR tgstation#62973 [ci skip]

* Give weight to childtype

* Reverts change I accidentally made to master, Lol

* Refactors Sign Language & Fixes Inconsistency (tgstation#62836)

Refactors Sign Language code so instead of copy-pasting the same giant wall of checks we can just use a proc.
Also now checks to see if your limb is disabled, which fixes people with disabled robotic limbs being able to sign still.
Finally, the tongue only has ORGAN_UNREMOVABLE if you attained it from the trait. I've been told that the tongue could be attained from meateors and I think that's funny as hell so I swapped that over.

* Automatic changelog generation for PR tgstation#62836 [ci skip]

* fixes empty defib hud (tgstation#62967)

The defib icon state was in between a bunch of antag hud icon states man it's not MY fault it's not with the REST of the medical huds

* Automatic changelog generation for PR tgstation#62967 [ci skip]

* Stun Batong now turns off/on constantly for short time after emp (tgstation#62943)

About The Pull Request
title

Why It's Good For The Game
i dunno, seems funny it gives more depth or something allows for more intresting interactions

Changelog
🆑
expansion: Stun Batong now turns off/on constantly for short time after emp

* Automatic changelog generation for PR tgstation#62943 [ci skip]

* Automatic changelog compile [ci skip]

* Yee-haw Update: Code Spaghetti Western Edition (tgstation#62937)

About The Pull Request

Previous PR (Closed Because I'm a Git Newbie)
https://youtu.be/AFa1-kciCb4

Ever wondered why we had cowboy boots in the autodrobe and clothesmate, but no hats?
Ever wanted to start a posse but the fluorescent lighting was beating down on your head?
Ever wanted to run your bar like an old-timey saloon but you were too ashamed of your bald spot?
Ever wanted to say "See you Space Cowboy" and actually mean it?

This is the PR for you!

image
Left to Right: Desperado Hat, Drifter Hat, Sheriff Hat (with Sheriff Vest), Ten-Gallon Hat, & Deputy Hat

All hats are craftable using two sheets of leather. The vest can be crafted with four sheets, and can be attached to jumpsuits.
The drifter hat and the ten-gallon can be found in the clothesmate. The others can be found in the autodrobe.
Every item has in-hand sprites!

This PR does not alter fugitive bounty-hunter items in any way.
Why It's Good For The Game

It always puzzled me that we had so many western-themed items, clothing, and references, but no hats to back it up. Now we can run cowboy gimmicks to our heart's content, and admins have more than enough to cobble together an event straight out of the old-space-west.
Changelog

cl
imageadd: Due to lobbying pressure from the Yee-haw Conservation Society, Nanotrasen vending machines now stock five new cowboy hats and an old-west vest.
/cl

* Automatic changelog generation for PR tgstation#62937 [ci skip]

* Adds an additional queue to the qdel subsystem to quickly filter out things that garbage collect within the first second. (tgstation#62969)

Before, all items deleted would sit in a queue for 5 minutes, with all shrinks and expansions of said queue requiring byond to copy all of these items over to the new list.

Theory: 99% of items soft-delete within byond within the first second. (5 minutes is only needed because a byond quirk with items referenced by verbs)

Result:

Within the first 7 minutes of a local test launch and round start, ~35,000 things get qdeleted.

Of those 35 THOUSAND things, only 12 things failed as still referenced with a 1 second pre-queue.

Said 12 things passed as garbage collected at the 5 minute queue.

(Note: 30 thousand of these items are from world start and round init.)

I have no data on how much this speeds anything up, leaving a 30 thousand list (that has to be copyed every time qdelete processes it and cuts off the items it processed) hanging around for no reason for the first 5 minutes of the round was all i needed to justify the pr.

* [PORT] Adds Safety Moth posters from Beestation! (Attempt 2) (tgstation#62936)

add: Added Safety Mothposters!

* Automatic changelog generation for PR tgstation#62936 [ci skip]

* Resprites (desouls) HoP's uniform (tgstation#62915)

imageadd: new and redone HoP uniforms

* Automatic changelog generation for PR tgstation#62915 [ci skip]

* Adds Supply Request Consoles to Cargo (tgstation#62860)

* Automatic changelog generation for PR tgstation#62860 [ci skip]

* fix harvest numbers (tgstation#62899)

* Automatic changelog generation for PR tgstation#62899 [ci skip]

* Bonfire can be made with wood (tgstation#62939)

Co-authored-by: Colovorat <lolqwert569@gmail.com>

* Automatic changelog generation for PR tgstation#62939 [ci skip]

* Forces areas to create their lighting objects when loaded from a map template (tgstation#62931)

Continues the journey of tgstation#61730. This time, all map templates will see their area lighting objects created upon getting loaded if they have static_lighting = TRUE. This means that places like Hilbert's hotel, as well as any map placed by an admin during the round, will now be able to properly have dynamic lights without any need for VV shenanigans.

Tested and working, I tested with Hilbert's hotel and by spawning a couple of other map templates to confirm.

Fixes tgstation#61558, closes tgstation#61340 (kinda not, but it's already fixed anyway, and loosely related to this) and should be the final nail in the coffin of tgstation#61349.

Courtesy ping to @TiviPlus, just in case you had anything to say about it.

* Automatic changelog generation for PR tgstation#62931 [ci skip]

* Washing machines can now shrink old t-shirts. (tgstation#62296)

About The Pull Request
image
Worn T-shirts, found in maintenance are now susceptible to shrinkage after repeated washing machine washings.
Too many machine washes can cause the shirts to shrink to the point that they simply disappear to the laundry dimension, as does real clothes in real life.

Why It's Good For The Game
Fun little hidden interaction with laundry machines due to my having noticed that laundry machines exist in the code still.

Changelog
🆑
expansion: Laundry machines can shrink particularly old, worn t-shirts. Remember kids, hand wash only.
/🆑

* Automatic changelog generation for PR tgstation#62296 [ci skip]

* Cleanup signals.dm (tgstation#62970)

* The bureaucratic mistake station trait can now truly chose any job. (tgstation#62960)

Too many Captains. So little time.

* Automatic changelog generation for PR tgstation#62960 [ci skip]

* Fixes a mistake that made space bartenders sad. (tgstation#62978)

* Fixes a mistake that made space bartenders sad.

* Update drinkingglass.dm

* Automatic changelog generation for PR tgstation#62978 [ci skip]

* Hair gradient preference no longer shows for species that don't have hair (tgstation#62950)

* Automatic changelog generation for PR tgstation#62950 [ci skip]

* Adds additional DB logging for admin help interactions (tgstation#62994)

Co-authored-by: Mothblocks <35135081+Mothblocks@users.noreply.github.com>

* Fixes some issues and an exploit with abandoned crates. (tgstation#62949)

* Automatic changelog generation for PR tgstation#62949 [ci skip]

* Possibly contains crude/indecent language (tgstation#62985)

* Automatic changelog generation for PR tgstation#62985 [ci skip]

* fixes random iceland turf in the middle of the bar on icebox (tgstation#62997)

* Automatic changelog generation for PR tgstation#62997 [ci skip]

* Automatic changelog compile [ci skip]

* space_heaters on process_atmos() (tgstation#62760)

Consistency between the various systems, space heater affected how atmos worked, should be processed on the same level (Maybe makes it operate twice as "strong"? but that's ok it's a bit bad now anyway -Lemon)

* Automatic changelog generation for PR tgstation#62760 [ci skip]

* Add Etheral lungs to the limbgrower (tgstation#62469)

Saturday morning, calm shift.
I spend most of my time staring at CANARY waiting for an atmos alarm to pop up while I sit at the foyer hearing what random craziness Poly is spewing and wondering about the previous and more interesting shifts.
When suddenly, an atmos alarm pops up at the Incinerator.
"Fucks sake, I bet the other atmos are torching the place again..."
I rush there expecting to fix a fire, but instead I find the start of something great...

The madman kidnapped an Ethereal to use as an organic Hydrogen farm, nothing I haven't seen before but... my hands start shaking, a spark of brilliance and potential in my eyes as I mumbled the words:
"I always hear that organic things are healthier... so what if we throw the Electrolyzer away and instead, print Etheral lungs in the limbgrower and implant them on monkeys..."
To our sadness, Ethereal lungs weren't added to the Ethereal disk design, this fixes that!

* Automatic changelog generation for PR tgstation#62469 [ci skip]

* DNA code now supports unique identity and features blocks of different size. (tgstation#62303)

tgstation#61980 (694c299) has uncovered a few issues with our dna datum code. Mainly the lack of support for ui/uf blocks of different sizes.
So, in order to fix this issues, I had to create four global lists, two for UI and UF blocks which sizes differ from DNA_BLOCK_SIZE and two more that store the points in the ui and uf strings where different dna blocks start (this way we avoid having to calculate them every getter and setter proc call).

This will add support for heterogeneous ui/uf block sizes and make 24-bit color features and identities possible.

To the maintainers of downstream codebases: Remember to add your modular code ui and uf color blocks to the identity_block_lengths and features_block_lengths global lists respectively ... Unless you have rewritten dna code on your end. If that's so, good luck.

Tested and working (unless CI says otherwise).

* Replaces Batform with Vampire houses (tgstation#62516)

Permissions for species change

image
About The Pull Request

Goodbye batform

Hello, new preference called vampire status. Outcast vampires act just like normal, Inoculated vampires join their department under a unified "vampire house name" everyone shares.

image

image

image
Why It's Good For The Game

I've heard people complain about batform for years now, if vampires aren't gone by next halloween we can at least enjoy making them far less griefy (ruining a lot of the fun of halloween as just a dumb grief holiday) and more roleplay oriented. I don't even know why vampires got a griefy spell when they already have their main mechanic encourage randomly attacking people and stealing their blood to stay alive
Changelog

cl
del: Batform is gone!
add: ...Replaced by vampire houses as a preference. Join your department as a vampire ménage!
/cl

* Automatic changelog generation for PR tgstation#62516 [ci skip]

* fix message spam for atmos construction (tgstation#62796)

About The Pull Request

partial fix to tgstation#62783
the check was never meant to have feedback messages added in it because it runs every UI tick. Added the message as additional requirement text.
GBP no update please
Why It's Good For The Game

fix
Changelog

cl
fix: fix message spam for manual pipe crafting
/cl

* Automatic changelog generation for PR tgstation#62796 [ci skip]

* Lavaland ash storms once again replenish ash on the ground. (tgstation#62954)

* Automatic changelog generation for PR tgstation#62954 [ci skip]

* Lets you build & deconstruct glass floors (tgstation#62890)

* Automatic changelog generation for PR tgstation#62890 [ci skip]

* Fixes Catwalk Tiles (tgstation#62988)

* Automatic changelog generation for PR tgstation#62988 [ci skip]

* A step up: The Lawyer Suit Resprite! (tgstation#62961)

Co-authored-by: Changelogs <action@github.com>

* Automatic changelog generation for PR tgstation#62961 [ci skip]

* Fix comms console not rejecting sending messages (tgstation#63008)

* Automatic changelog generation for PR tgstation#63008 [ci skip]

* Fix dragon rift not healing dragon (tgstation#62843)

About The Pull Request

Fixes tgstation#62816, caused by the healing aura refactor.
Changelog

cl
fix: Fix the space dragon's rift not healing the dragon.
/cl

* Automatic changelog generation for PR tgstation#62843 [ci skip]

* Move current pick_weight consumers to not pass in zeros (tgstation#62940)

About The Pull Request

pick_weight_allow_zero was a backwards compatible hack, so it exists alongside pick_weight.

I would really like to just remove the old proc and just rename it, so I want to log what, if anything, is relying on this behavior, to see if it's a bug or if it needs to be handled especially in that case.

* Add station trait for varying colored assistant jumpsuits (tgstation#62998)

About The Pull Request

Adds a new station trait to color the assistant jumpsuits semi-randomly.

There are several styles for it to choose from, including some especially colored ones, and some randomized ones. This includes the current alternative rainbow jumpsuits, but if GREY_ASSISTANTS is not enabled, then grey jumpsuits will be a valid random option.

I don't really want to hear too much more about the config, I got the OK on this from head admins.
Why It's Good For The Game

Adds more variety to look at on assistants, while still mostly preserving a consistent style if GREY_ASSISTANTS is enabled (as it is a station trait).
Changelog

cl
add: Added a station trait for varying colored assistant jumpsuits.
/cl

* Automatic changelog generation for PR tgstation#62998 [ci skip]

* Add disabling footstep sounds and parallax to the lag switch panel (tgstation#62999)

At the last Toolbox Tournament, @MrStonedOne pointed out some big performance consumers. Some are disablable through easy means, but others were not. This adds two easy ones to the mix.

* Automatic changelog generation for PR tgstation#62999 [ci skip]

* Makes hypnosis victims more visible to admins (tgstation#62745)

Gives hypnotized victims a visible antag hud for admins, puts them in the Check Antags panel, and makes their objectives visible in the Traitor Panel and on the round end report.
Why It's Good For The Game

Right now, victims of hypnosis are very difficult to visually identify as an admin. This not only distinguishes them easily but makes their objective easily visible too.
Changelog

cl
admin: hypnotized players now have a visible antag hud, are visible in the check antag panel, and have objectives visible on the traitor panel
expansion: hypnotized victims and their objectives are now visible on the round end report
/cl

* Automatic changelog generation for PR tgstation#62745 [ci skip]

* Add admin ckey to known alts panel (tgstation#62761)

About The Pull Request

Adds the admin who added the known alt to the known alts panel.
Changelog

cl
admin: The admin who adds a known alt is now logged in the table.
/cl

* Automatic changelog generation for PR tgstation#62761 [ci skip]

* I woke up to admins freaking out about massive lag spikes on manuel round 173107 (tgstation#62857)

They were seeing large amounts of hard deletes and weird spikes every ~2 seconds

Looking into the runtime log, the master controller was resetting every roughly 2 seconds
This was caused by an infinite loop in godslayer armor. Works like this

Player gets hurt enough to put them into crit, with enough brute to bring them up a stat level, and more hurt elsewhere
Godslayer armor detects harm, starts to heal
The heal heals the small amount of brute
This sends the stat update signal, which is sent before the stat is actually changed

The armor heals again, going through the same process
Any later calls to update_stat will send the same signal again because the signal is sent before the value is actually changed

This kills the master controller

I've fixed this by moving the cooldown reset, and by changing how stat is set in conjunction with the signal

* Automatic changelog generation for PR tgstation#62857 [ci skip]

* Makes sure the abandoned crates anti-tamper mechanism is reenabled only when the crate is locked. (tgstation#63012)

* Update interaction.dm (tgstation#62979)

* Automatic changelog generation for PR tgstation#62979 [ci skip]

* Vampire victims no longer receive messages about bloodsucking while unconscious. (tgstation#62953)

* Automatic changelog generation for PR tgstation#62953 [ci skip]

* Make varedited pumps/pipes keep their varedited part of their names (tgstation#62782)

* Make varedited pumps/pipes keep their varedited part of their names

* Make name override system still use initial

* Automatic changelog generation for PR tgstation#62782 [ci skip]

* de-hardcodes moth antennae preference icons (tgstation#62888)

It just makes the preferences actually appear for downstreams who have unique moth preferences.

* better logic for plural_s() (tgstation#62776)

makes it so that plural_s() actually follows most english rules
which is good
even though the proc is never used with this it might actually become useful (i have plans)

* Automatic changelog generation for PR tgstation#62776 [ci skip]

* Instant cameras can now be used as circuit shells. (tgstation#62810)

A circuit and shell components have been given to instant cameras around the station, so they can now be used for wiremod. There's no new wiremod design here since they can already be printed at any autolathe.
@Watermelon914 also told me a while ago that items should be given the shell component and not the usb port one.
Outside of the sci department and more specifically the protolathe and component printer, there's currently a lack of objects that can be used as circuit shells, which means wiremod is more or less limited to its own printables and the less versatile, higher end usb ports around the station. This is a small step toward a more inclusive wiremod that can use station devices for circuits building.

* Automatic changelog generation for PR tgstation#62810 [ci skip]

* Refactor cult dagger rune drawing and null rod rune clearing into components + minor rune / blood magic tweaks (tgstation#62918)

The act of inscribing runes, anchoring / unanchoring cult structures, deconstructing cult girders, and purging cultists of holy water have all been moved off of the cult dagger itself and onto a component, cult_ritual_item.

The act of destroying a cult rune or heretic rune with the null rod have been moved off of their respective items and onto the nullrod itself.

Added a signal, COMSIG_ITEM_ATTACK_EFFECT, for the above purposes.
Various cult spells and dagger interactions have had their code improved slightly.

* Automatic changelog generation for PR tgstation#62918 [ci skip]

* Department orders now include unique circuitboards (tgstation#62830)

About The Pull Request

I missed the circuitboards in the original pr, now they have em!
Why It's Good For The Game

Not really even printable right now but they are needed to reconstruct them in case of damage
Changelog

cl
fix: department order consoles include circuitboards
/cl

* Automatic changelog generation for PR tgstation#62830 [ci skip]

* Automatic changelog compile [ci skip]

* Adds note about IEChooser debugger to TGUI README (tgstation#63028)

* Fixes ghosts' boo ability going on cooldown when the light they tried to flicker didn't flick (tgstation#62965)

* Automatic changelog generation for PR tgstation#62965 [ci skip]

* Spades can now dig up botany soil plots on right click (tgstation#62957)

Co-authored-by: Mothblocks <35135081+Mothblocks@users.noreply.github.com>

* Automatic changelog generation for PR tgstation#62957 [ci skip]

* Make all slimes chirp like the non humans they are (tgstation#63000)

* Automatic changelog generation for PR tgstation#63000 [ci skip]

* Fix vampire clans applying to everyone (tgstation#63036)

* Automatic changelog generation for PR tgstation#63036 [ci skip]

* Fixes rites being able to be done by non-chaplains (tgstation#63002)

* Automatic changelog generation for PR tgstation#63002 [ci skip]

* Fixing parallax plane above blackness  (tgstation#62902)

* merge

* Update plane_master.dm

* What Iam doing

* Automatic changelog generation for PR tgstation#62902 [ci skip]

Co-authored-by: tgstation-server <tgstation-server@tgstation13.org>
Co-authored-by: tralezab <40974010+tralezab@users.noreply.github.com>
Co-authored-by: Mickyan <38563876+Mickyan@users.noreply.github.com>
Co-authored-by: Mothblocks <35135081+Mothblocks@users.noreply.github.com>
Co-authored-by: Iamgoofball <iamgoofball@gmail.com>
Co-authored-by: Watermelon914 <37270891+Watermelon914@users.noreply.github.com>
Co-authored-by: Watermelon914 <3052169-Watermelon914@users.noreply.gitlab.com>
Co-authored-by: itseasytosee <55666666+itseasytosee@users.noreply.github.com>
Co-authored-by: RandomGamer123 <31096837+RandomGamer123@users.noreply.github.com>
Co-authored-by: Ghom <42542238+Ghommie@users.noreply.github.com>
Co-authored-by: oranges <email@oranges.net.nz>
Co-authored-by: Emmanuel S. <emmanuelssr@gmail.com>
Co-authored-by: Thunder12345 <Thunder12345@users.noreply.github.com>
Co-authored-by: Thunder12345 <stewart@critar.demon.co.uk>
Co-authored-by: Changelogs <action@github.com>
Co-authored-by: esainane <esainane+github@gmail.com>
Co-authored-by: LordVollkorn <66637090+LordVollkorn@users.noreply.github.com>
Co-authored-by: RaveRadbury <3204033+RaveRadbury@users.noreply.github.com>
Co-authored-by: Krysonism <49783092+Krysonism@users.noreply.github.com>
Co-authored-by: UnokiAs <82665345+UnokiAs@users.noreply.github.com>
Co-authored-by: Kyle Spier-Swenson <kyleshome@gmail.com>
Co-authored-by: GoldenAlpharex <58045821+GoldenAlpharex@users.noreply.github.com>
Co-authored-by: Paxilmaniac <82386923+Paxilmaniac@users.noreply.github.com>
Co-authored-by: WarlockD <warlockd@gmail.com>
Co-authored-by: Ghilker <42839747+Ghilker@users.noreply.github.com>
Co-authored-by: Jackraxxus <60418544+Jackraxxus@users.noreply.github.com>
Co-authored-by: John Willard <53777086+JohnFulpWillard@users.noreply.github.com>
Co-authored-by: Archemagus <32466328+Archemagus@users.noreply.github.com>
Co-authored-by: Archanial <37456678+Archanial@users.noreply.github.com>
Co-authored-by: phasenoisepon <82134074+phasenoisepon@users.noreply.github.com>
Co-authored-by: Tim <timothymtorres@gmail.com>
Co-authored-by: ATH1909 <42606352+ATH1909@users.noreply.github.com>
Co-authored-by: Jeremiah <42397676+jlsnow301@users.noreply.github.com>
Co-authored-by: necromanceranne <40847847+necromanceranne@users.noreply.github.com>
Co-authored-by: twilightwanderer <88540658+twilightwanderer@users.noreply.github.com>
Co-authored-by: twilightwanderer <twilightwanderer@users.noreply.github.com>
Co-authored-by: Aleksej Komarov <stylemistake@gmail.com>
Co-authored-by: Tastyfish <crazychris32@gmail.com>
Co-authored-by: Farquaar <44147031+Farquaar@users.noreply.github.com>
Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
Co-authored-by: TacticalTortoise <87680969+TacticalTortoise@users.noreply.github.com>
Co-authored-by: BlueMemesauce <47338680+BlueMemesauce@users.noreply.github.com>
Co-authored-by: GuillaumePrata <55374212+GuillaumePrata@users.noreply.github.com>
Co-authored-by: Bond <58570888+TheBonded@users.noreply.github.com>
Co-authored-by: Fhaxaris <Chrisamcculler@gmail.com>
Co-authored-by: raffclar <gavxn@zanidrak.com>
Co-authored-by: Timberpoes <silent_insomnia_pp@hotmail.co.uk>
Co-authored-by: Zytolg <33048583+Zytolg@users.noreply.github.com>
Co-authored-by: vincentiusvin <54709710+vincentiusvin@users.noreply.github.com>
Co-authored-by: Burgermancoding <79515258+Burgermancoding@users.noreply.github.com>
Co-authored-by: unknown <burgercoding@gmail.com>
Co-authored-by: Colovorat <35225170+Colovorat@users.noreply.github.com>
Co-authored-by: OrionTheFox <76465278+OrionTheFox@users.noreply.github.com>
Co-authored-by: AxieTheAxolotl <86872535+axietheaxolotl@users.noreply.github.com>
Co-authored-by: Colovorat <lolqwert569@gmail.com>
Co-authored-by: ArcaneMusic <41715314+ArcaneMusic@users.noreply.github.com>
Co-authored-by: Nick <nick@nfreader.net>
Co-authored-by: Ryll Ryll <3589655+Ryll-Ryll@users.noreply.github.com>
Co-authored-by: Imaginos16 <77556824+Imaginos16@users.noreply.github.com>
Co-authored-by: dragomagol <66640614+dragomagol@users.noreply.github.com>
Co-authored-by: LemonInTheDark <58055496+LemonInTheDark@users.noreply.github.com>
Co-authored-by: Seris02 <49109742+Seris02@users.noreply.github.com>
Co-authored-by: ZeWaka <zewakagamer@gmail.com>
Co-authored-by: ShizCalev <ShizCalev@users.noreply.github.com>
Co-authored-by: Thedragmeme <81540056+Draggeru@users.noreply.github.com>
Co-authored-by: KIBORG04 <bossmira4@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Changes to functionality that modifies how effective certain methods are at powergaming
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet