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] Fix psyker echolocation lag #2443

Merged
merged 1 commit into from
Mar 17, 2024
Merged

[MIRROR] Fix psyker echolocation lag #2443

merged 1 commit into from
Mar 17, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1472
Original PR: tgstation/tgstation#82005

About The Pull Request

Fixes a slowly increasing lag for psykers, subtly introduced by #80042
Turns out the code expects all of the images to fully fade out before any new images start fading in, thus the expiry time + fade out time must not exceed the cooldown time.

Note that this will still be laggier than if you're NOT a psyker because of how things are rendered, but it should stay consistent.
Fixes #81446
Fixes #80442
Fixes #80798

Why It's Good For The Game

letting literally thousands of images build up on the client is not a good thing, turns out.

Changelog

🆑 FlufflesTheDog
fix: Psyker vision no longer causes infinitely worsening lag
/:cl:

* Fix psyker echolocation lag (#82005)

## About The Pull Request
Fixes a slowly increasing lag for psykers, subtly introduced by #80042
Turns out the code expects all of the images to fully fade out before
any new images start fading in, thus the expiry time + fade out time
must not exceed the cooldown time.

Note that this will still be laggier than if you're NOT a psyker because
of how things are rendered, but it should stay consistent.
Fixes #81446
Fixes #80442
Fixes #80798
## Why It's Good For The Game
letting literally thousands of images build up on the client is not a
good thing, turns out.
## Changelog
:cl:
fix: Psyker vision no longer causes infinitely worsening lag
/:cl:

* Fix psyker echolocation lag

---------

Co-authored-by: FlufflesTheDog <[email protected]>
@ReezeBL ReezeBL merged commit 8e104b7 into master Mar 17, 2024
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-1472 branch March 17, 2024 00:22
AnywayFarus added a commit that referenced this pull request Mar 17, 2024
Iajret pushed a commit that referenced this pull request May 13, 2024
* Food vendors actually drop their appropriate parts

* null abstract object's icon state

* subtypesof is a thing that exists

Co-authored-by: Bloop <[email protected]>

---------

Co-authored-by: Bloop <[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