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

Bottom deck AI core gone #17744

Conversation

DreamySkrell
Copy link
Contributor

image
image

it served no use, only taking up space

@github-actions github-actions bot added Changelog Required 🗺️ Mapping - Horizon The PR touches the Horizon map files. labels Nov 7, 2023
@DreamySkrell
Copy link
Contributor Author

!review

@FluffyGhoster
Copy link
Contributor

That's where the AI shell is located at roundstart, I do not see it on the mapdiff, where did you move the AI shell to?

@DreamySkrell
Copy link
Contributor Author

That's where the AI shell is located at roundstart, I do not see it on the mapdiff, where did you move the AI shell to?

image

idk why mapdiffbot is not rendering doors

@FluffyGhoster
Copy link
Contributor

That's where the AI shell is located at roundstart, I do not see it on the mapdiff, where did you move the AI shell to?

image

idk why mapdiffbot is not rendering doors

Ah, I see it now

I do not think that's a good placement, it's the easiest path for someone to get in and card the AI, and it would be blocking the door unless the AI moves it or it gets destroyed; I am also not sure why there's a suit cycler in this room that's supposed to only have synthetics inside, perhaps remove the suit cycler and put the shell there?

@DreamySkrell
Copy link
Contributor Author

DreamySkrell commented Nov 7, 2023

That's where the AI shell is located at roundstart, I do not see it on the mapdiff, where did you move the AI shell to?

image
idk why mapdiffbot is not rendering doors

Ah, I see it now

I do not think that's a good placement, it's the easiest path for someone to get in and card the AI, and it would be blocking the door unless the AI moves it or it gets destroyed; I am also not sure why there's a suit cycler in this room that's supposed to only have synthetics inside, perhaps remove the suit cycler and put the shell there?

image

it's not a suit cycler it's the robot storage thingy they use to cryo 😩

moved the shell to this spot here

@SleepyGemmy
Copy link
Contributor

the PR touches the shuttles. is that intentional?

@DreamySkrell
Copy link
Contributor Author

the PR touches the shuttles. is that intentional?

that is not intentional, and honestly no idea how that even happened

some kind of bad merge? idk.......

thank you for catching it

@DreamySkrell
Copy link
Contributor Author

image
image

these are the only changes according to mapdiffbot now so it should be good (I hope)

@Alberyk Alberyk added this pull request to the merge queue Nov 13, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 13, 2023
@Arrow768
Copy link
Member

i am a bit unsure as that makes it a bit easier to blow up the Ai with a bomb from below.
But I guess we´ll see if that becomes a pattern

@Arrow768 Arrow768 added this pull request to the merge queue Nov 20, 2023
Merged via the queue into Aurorastation:master with commit 4201960 Nov 20, 2023
19 checks passed
@DreamySkrell DreamySkrell deleted the 2023-11-03--mainly-bottom-deck-ai-gone branch January 2, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Merge 🗺️ Mapping - Horizon The PR touches the Horizon map files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants