cmss13-devs / cmss13

Contains the code for CM-SS13
https://cm-ss13.com
GNU Affero General Public License v3.0
96 stars 545 forks source link

Motion Detector Working in Satchel #1078

Closed IMEI21 closed 1 year ago

IMEI21 commented 2 years ago

Testmerges:

Description

MD works in satchel

Difference between expected and actual behavior

MD should turn off when placed in satchel, confirmed this with ahelp.

Steps to reproduce

Open satchel. Place MD in your hand. Turn on MD. Place MD inside satchel. Leave satchel open. MD still works.

Issue bingo (not required to be all checked)

silencer-pl commented 2 years ago

If memory serves, while it does still work, it will not display the directional blips on the game screen, will only beep and indicate the amount of contacts, which I always thought was intended?

IMEI21 commented 2 years ago

I was still seeing blips and numbers on the MD when I stuck it in my satchel.

On Fri, Sep 16, 2022 at 10:00 PM silencer-pl @.***> wrote:

If memory serves, while it does still work, it will not display the directional blips on the game screen, will only beep and indicate the amount of contacts, which I always thought was intended?

— Reply to this email directly, view it on GitHub https://github.com/cmss13-devs/cmss13/issues/1078#issuecomment-1249985170, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3DY7MGTBVJT5HW7H6NN2C3V6UX6VANCNFSM6AAAAAAQOZLAVA . You are receiving this because you authored the thread.Message ID: @.***>

lajolico commented 1 year ago

I cannot reproduce this bug for the life of me.

lajolico commented 1 year ago

@memesky117 Were you able to?

ThePiachu commented 1 year ago

I think it works as intended. You turn on the MD, put it in the satchel, it beeps and tells you the number, but doesn't show the blips. Useful early warning system, not completely OP.

Remember guys - it used to be that you could yeet the MD on the floor and it would still work, so it's still reasonable.

IMEI21 commented 1 year ago

Ok, that seems to make sense. I just didn't want to be using it in an unfair way when I discovered it.

Thank you so much everyone for looking into it for me.

On Sun., Nov. 13, 2022, 03:16 ThePiachu, @.***> wrote:

I think it works as intended. You turn on the MD, put it in the satchel, it beeps and tells you the number, but doesn't show the blips. Useful early warning system, not completely OP.

Remember guys - it used to be that you could yeet the MD on the floor and it would still work, so it's still reasonable.

— Reply to this email directly, view it on GitHub https://github.com/cmss13-devs/cmss13/issues/1078#issuecomment-1312681980, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3DY7MHLVQUK5WBW2O4XJY3WICWWZANCNFSM6AAAAAAQOZLAVA . You are receiving this because you authored the thread.Message ID: @.***>

AtomAoim commented 1 year ago

Intended to still work but not see locations. Unable to replicate. Reopen if it occurrs again.