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

Zone de commandement général - 2 - Mapping CIC Rework / Part 2 #4251

Closed
wants to merge 167 commits into from

Conversation

LC4492
Copy link
Contributor

@LC4492 LC4492 commented Aug 24, 2023

About the pull request

This PR is focused on updating and modifying the Bridge and any command rooms around it (Such as the XO's/CO's Quarters, Synthetic Storage, etc), with the purpose of giving the place a refreshment and, of course, an improvement on efficiency. Feel free to analyse the photos below and to tinker with the map files. PS: This is an updated re-post of the last PR, carrying now a more utilitarian design, with a smaller bubble compartment and closer Overwatch computers. All changes requested on the last PR have been added here, including a new hall between the conference room and...Jones bed.

Explain why it's good for the game

Analyse the map changes and say if its an improvement or not.

Testing Photographs and Procedure

Captura de Tela (445)

image

image

Changelog

🆑
add: Added a little more munitions for the MK1 lockers (2 more normal mags per locker) and some extra boxes of buckshot/slugs to the MK221s lockers. Added a red-alert closet on the MP Checkpoint, of which holds a M39 sub-machinegun.
maptweak: Changed the Almayer's Bridge and the rooms surrounding it, such as the PO's, SO's, XO's and CO's quarters and others, with the help of Wulferion, thank you.
/:cl:

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Oct 23, 2023
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Oct 26, 2023
@cm13-github
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Oct 27, 2023
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@Yellows91
Copy link

Its good but the SOs OW is too tight.

1x1 will have you run into the SO or have to go around (as XO/CO this is time critical that you cannot afford to go around)
2x2 will be better if possible

@LC4492
Copy link
Contributor Author

LC4492 commented Oct 29, 2023

Its good but the SOs OW is too tight.

1x1 will have you run into the SO or have to go around (as XO/CO this is time critical that you cannot afford to go around) 2x2 will be better if possible

I've given the bubble three exits, two of which allow for you to enter and exit without going around the SOs and/or running into them. Making the area bigger is still in prototype.

@ghost
Copy link

ghost commented Oct 29, 2023

These edits deleted/remove the ASO spawn. Before they would wake up with the rest of CIC, now they wake up with general crew/privates.

@LC4492
Copy link
Contributor Author

LC4492 commented Oct 30, 2023

These edits deleted/remove the ASO spawn. Before they would wake up with the rest of CIC, now they wake up with general crew/privates.

Yes, the cryo room didn't had enough space for the ASO, so its still pending to select on where he will be sitting next.

@Zonespace27
Copy link
Contributor

Thank you for the effort you've put into this PR, but it will be closed.

There are some good ideas here, but there's a lot of things that I and other maintainers disagree with on a conceptual level. In the future, please atomize your PRs more than this (this touches a hell of a lot more than CIC) so that some changes can get in.

Ultimately, I don't think this is much of an improvement over the old CIC, and if it's not broken, then there's nothing to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge Mapping did you remember to save in tgm format?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants