Skip to content
This repository has been archived by the owner on Jul 27, 2021. It is now read-only.

[MIRROR] [PORT] Emissivity Blockers #254

Merged
merged 2 commits into from
Apr 29, 2021
Merged

[MIRROR] [PORT] Emissivity Blockers #254

merged 2 commits into from
Apr 29, 2021

Conversation

ghost
Copy link

@ghost ghost commented Apr 26, 2021

Original PR: BeeStation/BeeStation-Hornet#3993

About The Pull Request

Includes:
tgstation/tgstation#49454
tgstation/tgstation#57934

Why It's Good For The Game

Emissivity is now clipped instead of showing over every single thing in the game.

Changelog

馃啈 actioninja, Kylerace
fix: Glowing overlays no longer render above everything, and mobs and items properly block them
/:cl:

francinum and others added 2 commits April 26, 2021 02:26
* Better glowing lights (#49454)

* better glowy lights

* phase 2

* some doc comments

* working emissive blocking

* flag so these rotate correctly

* full brightness screens

* doc comments and rename

* better doc comments

* stub overrides

* better autodoc format

* Update code/game/atoms_movable.dm

Co-Authored-By: Emmett Gaines <ninjanomnom@gmail.com>

* Update code/modules/lighting/emissive_blocker.dm

Co-Authored-By: Emmett Gaines <ninjanomnom@gmail.com>

* render target defines for all planes

* update proc for emissive blocker

Co-authored-by: Emmett Gaines <ninjanomnom@gmail.com>

* emissive blockers are now just an overlay (with kickass fucking graphs). FUCK YOU MAPTICK (#57934)

currently emissive blockers are objects added to the vis_contents of every item. however mutable appearances as overlays can serve this role perfectly well and according to my tests should cause less maptick per item. for mobs mutable appearances dont work for reasons i dont understand so instead this adds the em_block object to overlays instead of vis_contents. these both use atom/movable/update_icon() now

Co-authored-by: Rob Bailey <actioninja@gmail.com>
Co-authored-by: Emmett Gaines <ninjanomnom@gmail.com>
Co-authored-by: Kylerace <kylerlumpkin1@gmail.com>
@Vexylius Vexylius merged commit e54af38 into master Apr 29, 2021
github-actions bot pushed a commit that referenced this pull request Apr 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants