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

Emergency shutters dont work #4880

Open
3 tasks done
Releasethesea opened this issue Nov 8, 2023 · 9 comments
Open
3 tasks done

Emergency shutters dont work #4880

Releasethesea opened this issue Nov 8, 2023 · 9 comments
Labels
Confirmed Bug We have the same at home!

Comments

@Releasethesea
Copy link
Contributor

Testmerges

#4475, #4812, #4793, #4866, #4863, #4872

Round ID

19438

Description of the bug

Emegency shutters are opening in the wrong areas all over the ship

What's the difference with what should have happened?

they should only open where they are at, not outside of the area, heres my crude drawings showing what happens, this occurs are multiple locations through the ship which it shouldnt be doing

fireshutter2
fireshutter

How do we reproduce this bug?

  1. toggle the fire alarm
    2.look outside of the area
    3.see its not properly aligned with the area/where its ment to be
    ...

Issue Bingo

@Releasethesea Releasethesea added the Bug Admin events aren't bugs label Nov 8, 2023
@Huffie56
Copy link
Contributor

Huffie56 commented Dec 9, 2023

after testing this is a bug and not a simple mapping issue.

@Huffie56 Huffie56 added Confirmed Bug We have the same at home! and removed Bug Admin events aren't bugs labels Dec 9, 2023
@ghost
Copy link

ghost commented Dec 27, 2023

yeah, this is fucked.

@Huffie56
Copy link
Contributor

so yes commenting out looking out work for some situation but on other we need code to look out for adjacent shutter to close.
image
let's imagine an extreme example where red is "area A"(red)and is surrounded by other areas with their shutter at their access
@Cthulhu80

@Huffie56
Copy link
Contributor

Huffie56 commented Dec 28, 2023

you need in some way to only restrict it to only check adjacent area that have floor on both area.
not sure if that phrase make sense....

@Huffie56
Copy link
Contributor

Huffie56 commented Dec 28, 2023

remapping area could also be a solution in most case...?

@ghost
Copy link

ghost commented Dec 28, 2023

wat

@ghost
Copy link

ghost commented Dec 28, 2023

I mean, idk man, whatever ya want me to do. I ain't no mapper though. I think you are, right? Might be a task for you then.

@Huffie56
Copy link
Contributor

I mean, idk man, whatever ya want me to do. I ain't no mapper though. I think you are, right? Might be a task for you then.

let's just wait to see what a maintainer say about your solution.

@ghost
Copy link

ghost commented Dec 28, 2023

Said this on discord but, maybe instead we can instead have it so that all connected fire shutters to any adjacent shutter close as well. That way it doesn't look funky and closes all the nearby room shutters.

@Drulikar Drulikar linked a pull request Dec 28, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Confirmed Bug We have the same at home!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants