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

Adds Sensor Towers To All Maps #4416

Closed
wants to merge 6 commits into from

Conversation

Steelpoint
Copy link
Contributor

@Steelpoint Steelpoint commented Sep 15, 2023

About the pull request

Adds sensor towers to all maps.

The towers are placed in generally xeno controlled areas, but not too far back to be impractical for Marines to take and hold,

Explain why it's good for the game

Consistency.

Furthermore, sensor towers give both sides a general area to fight over. The current towers are generally in xeno controlled areas but are forward enough to encourage both sides to realistically fight over it.

Furthermore this should help in Marine end-games by letting them track the last remaining xenos faster instead of relying on blind searching.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
mapadd: Sensor Towers have been added to all maps.
/:cl:

@github-actions github-actions bot added the Mapping did you remember to save in tgm format? label Sep 15, 2023
@realforest2001 realforest2001 added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Sep 15, 2023
@morrowwolf
Copy link
Member

Haven't forgotten this but I'm thinking about it.

Copy link
Member

@morrowwolf morrowwolf left a comment

Choose a reason for hiding this comment

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

image

These warning stripes on Solaris should be contiguous so like remove them from the pathway.

Fiorina looks fine.

Remove Shivas changes as that whole map is being reworked right now.

Put LV624's tower somewhere in the lake area. Everywhere else it's like smack dab in the middle of a hive and once things starting going marine favored the lake area is usually opened up anyhow just by how the map plays.

New varadero move it north to here-ish:
image

Again we don't really want it right outside a hive location like that.

Soro: image

You'll want to move it off the wall as it's clipping through pretty bad here.

Otherwise, I think we'll go ahead with this idea. Objectives gameplay is a goal.

@morrowwolf morrowwolf marked this pull request as draft September 26, 2023 08:34
@cm13-github cm13-github added the Merge Conflict PR can't be merged because it touched too much code label Sep 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 Sep 26, 2023
@cm13-github
Copy link
Contributor

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

@Steelpoint Steelpoint marked this pull request as ready for review September 26, 2023 12:12
@Steelpoint
Copy link
Contributor Author

Changes made

Copy link
Member

@morrowwolf morrowwolf left a comment

Choose a reason for hiding this comment

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

Looks like some accidental changes left on New Varadero and LV624

@morrowwolf morrowwolf marked this pull request as draft September 26, 2023 12:57
@Steelpoint
Copy link
Contributor Author

I'll do a new PR for the change since I hate map merge conflicts.

@Steelpoint Steelpoint closed this Sep 26, 2023
github-merge-queue bot pushed a commit that referenced this pull request Sep 27, 2023
# About the pull request
Continuation of #4416 

Adds sensor towers to all maps.

All requested changes have been made

# Explain why it's good for the game
# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

Put screenshots and videos here with an empty line between the
screenshots and the `<details>` tags.

</details>


# Changelog
:cl:
mapadd: Sensor Towers have been added to all maps.
/:cl:

Co-authored-by: Steelpoint <[email protected]>
@Steelpoint Steelpoint deleted the sensormap branch July 17, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Mapping did you remember to save in tgm format?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants