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

[READY] [AGONIZING] [IDB IGNORE] [MDB IGNORE] The final merge #58

Merged
merged 1,127 commits into from
Jul 31, 2021
Merged

Conversation

TheChosenEvilOne
Copy link
Collaborator

@TheChosenEvilOne TheChosenEvilOne commented Jul 27, 2021

agony

Wayland-Smithy and others added 30 commits July 16, 2021 19:55
…60191)

Admin Supplypod Presets can now also be loaded by double clicking their label.
… terrible (tgstation#60176)

Yes apparently you can code fluff paper badly. Whoever updated it to the supermatter version did the update in the map file for some reason. Fixes tgstation#60171.
A Charlie station fluff paper about the Supermatter is no longer misnamed "singularity generator". THERE WAS NEVER SUCH A THING AS A SINGULARITY GENERATOR. OCEANIA HAD ALWAYS BEEN AT WAR WITH EASTASIA.
This PR improves item action button updating code by introducing a proc to update all item's associated action buttons, which simplifies the process and reduces code duplication.

I also fixed a relevant bug where a PDA's action button icon did not properly update upon light toggle because it did not force the button update. Forcing the button update is needed in this case due to how light overlay operates, where the button icon state stays the same, so the code thought updating the action button icon was unnecessary.
This shifts the position of the toxins airlock control by one tile so that it can be accessed from inside the airlock chamber. This is applied to Tram, Delta, Kilo, and Meta. Only Ice was properly setup.

Also kills a rogue decal inside a medbay wall on Delta.

Thanks to Adhoc on discord for reporting this.
* Only blend areas when it's actually needed

* Respect varedits
…ptimization (tgstation#60219)

fixes tgstation#60197
woke up today with a ridiculous idea of semi-automatic compile time loop unrolling, wasnt worth the complexity in the least but it made the basis of this PR which i then continued work on. makes area_sensitive_contents into a more general system of important_recursive_contents where we can define reasonable uses to replace recursive contents iteration of the type found in get_hearers_in_view() as long as everything that uses it isnt something incredibly common to the point that it noticeably increases memory usage.
Adds the module component that is basically a subroutine. Allows you to compact your logic into a bunch of functions.
…lows scanning ids on unregistered circuits through shells. (tgstation#60227)

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

Previously what it would do is create up to three objects; a cone, a lighting image and a holder
Then whenever the lighting image was placed into the holders vis contents and the holder would be teleported several times when the parent would to turn or move, and the cone would be spun when the parent spun

Instead of doing that it now just translates a single overlay every time that something is updated
Maintenance loot spawners now fire at roundstart, rather than during the initialization of the Atoms subsystem.
Maintenance loot spawners do not qdel themselves during the spawning process, but persist, and merely hide themselves from living players.

As a cute side effect, this lets preround observers look at the colourful maintenance loot dice icons before the round starts. Observers can continue to see the loot dice post-round, but they are deliberately
very faint.
… no bomb resist (tgstation#60261)

Co-authored-by: Watermelon914 <3052169-Watermelon914@users.noreply.gitlab.com>
@TheChosenEvilOne TheChosenEvilOne added Merge Conflict WE HATE CONFLICTS feature good shit fix also nice readds fun adds fun back into the game to be defenestrated someone made a fucking oopsy waiting for upstream merge DO NOT MERGE Don't merge this PR or you will face the wrath of cooders ✔️ Wedditor TEST YOUR SHIT labels Jul 27, 2021
@Djiq
Copy link
Collaborator

Djiq commented Jul 27, 2021

only a few issues:agony:

@Djiq
Copy link
Collaborator

Djiq commented Jul 29, 2021

you should be able to fix all dmi conflicts using that one tool it is really good

@Djiq
Copy link
Collaborator

Djiq commented Jul 29, 2021

tools > dmi > Resolve Merge Conflicts
should resolve all icon ones at least.

@TheChosenEvilOne TheChosenEvilOne removed the Merge Conflict WE HATE CONFLICTS label Jul 31, 2021
@TheChosenEvilOne TheChosenEvilOne removed the DO NOT MERGE Don't merge this PR or you will face the wrath of cooders label Jul 31, 2021
@TheChosenEvilOne TheChosenEvilOne changed the title [IDB IGNORE] [MDB IGNORE] The final merge [READY] [AGONIZING] [IDB IGNORE] [MDB IGNORE] The final merge Jul 31, 2021
@Djiq
Copy link
Collaborator

Djiq commented Jul 31, 2021

based and redpilled i approve

@Djiq Djiq merged commit 978b2dd into master Jul 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature good shit fix also nice readds fun adds fun back into the game to be defenestrated someone made a fucking oopsy ✔️ Wedditor TEST YOUR SHIT
Projects
None yet