brings us up slightly to tg at the end of october (like 3 days) [SQUASH AND MERGE IF YOU WANT TO DIE] - #1199
Open
carda-sometimesknownas-paxilmaniac wants to merge 40 commits into
Conversation
## About The Pull Request Tin, the current algorithm is very greedy and can call Enter() on thousands of turfs potentially depending on how your map is set up. This really isn't necessary and leads to spess lag and sometimes spess crashes. Instead of doing that we can test a smaller random subset of tiles with Enter() and achieve basically the same results. This basically just puts a safe bound on how many Enter() calls we can do. <img width="1307" height="739" alt="dreamseeker_6nE6Hjad7B" src="https://github.com/user-attachments/assets/3405ee8f-1af8-4dce-b253-1fcbf47412ea" /> ## Why It's Good For The Game Less spess lag ## Changelog :cl: code: optimizes space vine spawning to create less lag. /:cl: --------- Co-authored-by: Ghom <42542238+Ghommie@users.noreply.github.com>
Co-authored-by: Fghj240 <fakeemail@notrealemail.com> Co-authored-by: necromanceranne <40847847+necromanceranne@users.noreply.github.com>
## About The Pull Request Fixes tgstation#93589, fixes tgstation#93594, fixes (probably) tgstation#93663 by BANNING mobs that have things buckled on them from getting on. also if you try to buckle to something that is already in the cart you CANT do that anymore ## Why It's Good For The Game It fixes golf cart bugs which are NOT good.. :( ## Changelog :cl: fix: fixes permanent soul binding between golf carts and riders /:cl:
## About The Pull Request This was only set on the cat ears subtype but I think it should probably be set on the parent type ## Why It's Good For The Game Child types should all have correct abstract type
## About The Pull Request Vampire blood was changed from `U` to a new `V` type but they didn't update the `V` type to be universal like the `U` type ## Changelog :cl: Melber fix: Vampires suck again, I mean, they can suck any bloodtype again /:cl:
…n#93698) ## About The Pull Request Title. ## Why It's Good For The Game These technically aren't weapons, they're shoe shoes silly. But also the non-capital at the start annoyed me. ## Changelog :cl: spellcheck: Clown Op shoe description no longer starts with a lower case. /:cl:
## About The Pull Request Fix unused imports in AI display picker files and unnecessary fragment in newspaper file ## Why It's Good For The Game Lint warning fix ## Changelog No
…ilo (tgstation#93681) ## About The Pull Request - What it says above, makes them drop materials instead of sending them to the shadow realm - Replaces some 1 argument `round()'s` with `floor()'s` in material logic, someone should regex all of them sometime for a giant conflict merge PR ## Why It's Good For The Game > What it says above, makes them drop materials instead of sending them to the shadow realm - Makes custom-made refinery setups a bit more viable, alongside recovering from the ore silo being bombed/eaten by a hungry boi > Replaces some 1 argument `round()'s` with `floor()'s` in material logic - Round() using 1 argument is deprecated, should slowly start switching over ## Changelog :cl: fix: the ore refineries now drop stacks of materials below them if they are not linked to a silo instead of being sent to the shadow realm /:cl:
## About The Pull Request I had the idea to dump all of `as JOB` and `as ANTAG` into sperate string lists and increase the weight of the as antag ones based on station threat level. Sorted out the strings to make sure I was not gonna miss any. Need feedback on that idea though. ## Changelog :cl: spellcheck: Lawyer and Coroner round tips now reflect how many job slots they have. /:cl:
## About The Pull Request Missing these two causes a runtime whenever you spawn in, and certain verbs won't work without them (spawn debug full crew won't work without an observer landmark) ## Why It's Good For The Game Make absolute minimum define more usable ## Changelog N/A
## About The Pull Request QDEL_NULL for the mook bard's `held_guitar` ## Why It's Good For The Game Some kind of hard delete waiting to happen ## Changelog N/A
## About The Pull Request title ## Why It's Good For The Game idk ## Changelog :cl: spellcheck: Adds missing space in gullion description. /:cl:
## About The Pull Request Most screen alerts that use the midnight hud style no longer have the button baked in their icon. Other screen alerts with their own background or shape (robot and mech alerts, atmos, heretic buffs or debuffs etc.) are not affected. Also updated a couple sprites but didn't spend too much time on them. Mostly reusing existing assets. Montage of how the alerts look on threee different hud styles (Operative, Trasen-Knox, Detective, ALSO I FIXED THE BUCKLED ALERT ALREADY): <img width="293" height="323" alt="image" src="https://github.com/user-attachments/assets/3a2b972b-aa5a-4c27-a454-c8c39acf6e20" /> It looks only a smidge iffy on the syndicate since the top and bottom borders aren't layered over all the overlays, but it isn't something to worry about in this PR. ## Why It's Good For The Game Screen alerts always had the midnight hud button baked in their icon states (now overlays), which completely disregard the player's hud setting, much unlike action alerts buttons. Melbert has also said that it'd be nice if the code for action buttons could also be used in screen alerts and viceversa, to slim things down. That's obviously not what I'm doing today, but having most of the screen alerts already without the baked background will surely help if we ever pursue that objective. ## Changelog :cl: refactor: Refactored screen alerts a little. Most should now fit the player's hud style. Report any issue. imageadd: A few screen alerts have been polished/updated a little. /:cl:
Ops found that manually rebooting the server skips config syncs. Rectify --------- Co-authored-by: Jordan Dominion <Cyberboss@users.noreply.github.com> Co-authored-by: Ghom <42542238+Ghommie@users.noreply.github.com>
## About The Pull Request Despite what the code comments said we can rename them and we should because they make no sense ## Why It's Good For The Game The save file keys for cat features should have cat in the name and not human ## Changelog :cl: server: renamed feature_human_ears and feature_human_tail to feature_cat_ears and feature_cat_tail respectively /:cl:
## About The Pull Request One underline ## Changelog :cl: Melbert fix: Fix open wing sprite /:cl:
## About The Pull Request Fun idea discussed on the discord. At low pressure, cans of soda will burst at 67.458 kPa, which is WAY higher than I would have expected, but that may just say something about the construction of the average can of soda. ## Why It's Good For The Game Fun little realism feature, which I found amusing enough for a quick 15 minute project. Only applies to cans that are actually exposed to atmos, so a held can while in space or in the bag won't trigger this reaction. ## Changelog :cl: add: Leaving a can of soda exposed to the elements at low pressure will now burst. /:cl: --------- Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
…tgstation#93650) ## About The Pull Request Baby's first refactor! Originally meant for a downstream, I was told to PR it here instead. Please do tell me if I did something wrong. Previously the Lasertag guns would directly check if you were wearing a vest before dealing stamina damage. Now they instead check for a component and it's team, which is added by the vest. ## Why It's Good For The Game This should make it easier to add new lasertag equipment. It compiles and lasertag functionality seems to remain unchanged from the player side.
carda-sometimesknownas-paxilmaniac
requested review from
00-Steven,
TheOneAndOnlyCreeperJoe,
dawsonkeyes,
desertmote and
kannthus
as code owners
September 4, 2026 05:53
github-actions
Bot
added
Map Edit
carda-sometimesknownas-paxilmaniac
changed the title
|
This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
16 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
About The Pull Request
see title
Why It's Good For The Game
I plan to work on updates whenever I have time to do them, like a few weeks to a month at a time for my sake, this brings us to the actual last day of the month