Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@Notamaniac
Copy link
Contributor

Document the changes in your pull request

Closes #12380

If two people are buckled next to each other, the mirror shield will bounce off each other indefinitely, causing massive damage. This will still happen if the victims are cuffed and buckled, but eh.

Wiki Documentation

None

Changelog

🆑
tweak: The mirror shield will attempt to unbuckle the victim it's thrown at, then launch itself. This stops the mirrorshield from insta-killing two people buckled together, however if they're bucklecuffed it's still instadeath.
/:cl:

@Yogbot-13 Yogbot-13 added the Tweak This PR contains changes to the game. label Jan 9, 2022
@Reed0506
Copy link

Reed0506 commented Jan 9, 2022

i feel a better solution would be to set a hard limit on how many times it can bounce

@Notamaniac
Copy link
Contributor Author

i feel a better solution would be to set a hard limit on how many times it can bounce

That might creep into balance territory, and wouldn't solve the core issue of buckling two people together and getting in multiple hits on one person, even if the hard limit was set down to 3/4. (Right now recursion code stops the bounces at 8 soo...)

The another solution might be to keep a var tracking the last two people the shield hit, and if the shield were to target one of those people, stop bouncing. But that might run into some edge cases (what if 3 people are in the same area?)

Alternatively the shield could keep the list of people it hit and work off that, but I don't know enough byond code to implement that solution cleanly

@Hopekz
Copy link
Contributor

Hopekz commented Jan 10, 2022

Why does this need a solution? It's very situational and when it happens it's more of a "cool" thing

@Notamaniac
Copy link
Contributor Author

Why does this need a solution? It's very situational and when it happens it's more of a "cool" thing

Eh someone made an issue about it, and it's not very cool to be on the receiving end of this

@UselessTheremin
Copy link
Contributor

this is definitely a weird solution yeah why not just have limited bounces

Copy link
Contributor

@JamieD1 JamieD1 left a comment

Choose a reason for hiding this comment

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

Sounds like its been abused by a few people. Good enough solution.

@JamieD1 JamieD1 merged commit c91fa6f into yogstation13:master Jan 12, 2022
Yogbot-13 added a commit that referenced this pull request Jan 12, 2022
@Notamaniac Notamaniac deleted the mirrorshield branch January 17, 2022 20:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Tweak This PR contains changes to the game.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Noncultists throwing mirror shields can kill people who are next to each other instantly

6 participants