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

Sweeping LV522 Changes + FORECON GL replaced by FORECON Sniper (Without the sniper) #3917

Merged
merged 18 commits into from
Jul 28, 2023

Conversation

spartanbobby
Copy link
Contributor

@spartanbobby spartanbobby commented Jul 18, 2023

About the pull request

This PR makes alot of small changes to LV522 along with alot of big ones mainly

Blocks off the area north of fitness to prevent a "just go north" meta
Adds a new more central way to the reactor that branches off to 4 flanks should give LZ2 more use since it's a bit more central than LZ1
Adds a new route into the reactor from the SE
Blocks off the small alleyway north of the engineering funnel people into the dorm building
tightens up some parts of the reactor loosens some other parts
removes some detailing in the main hive areas to help builders
Adds new secrets to be found on the map
adds an overextension flank for those very brave marines to use
Changes to LZ2 making it more compressed
Addition of the FORECON Sniper (replacing the FORECON GL)
the FORECON sniper is a unique take on FORECON he spawns away from his fellow survivors next to the corpse of his battle buddy the advantage given to the sniper however is his thermal tarp a subtype of the ghillie suit with a placeholder sprite at the moment that will allow the sniper to hide and stealthily move throughout the colony to hopefully regroup with his squad
the M42A rifle has been removed from the map
Adds a small piece of text explaining that FORECON should probably move west to the crashed dropship to the FORECON intro
swaps the black weedable turf with a browner one for people who have bad eyes and were mistaking it for weeds
swaps the assignments for defines bc nanu told me someone would ask me to do it at somepoint
gives FORECON radioheadsets with the SOC and common channel because the desc literally says FORECON use it and it doesn't make sense for them to be radio-less

Explain why it's good for the game

Map stuff:
LV522 currently just has a "go north" problem this PR aims to fix that by opening more routes for the marines to go, centralize the main route in an attempt to see more use in the rest of the colony and give LZ2 more use I'd really like to see this test-merged for a few days so see what needs addressing in this new area

the wall north of engineering looks kinda funky but having it there funnels people into the building and into the eastern section of the colony rather than a straight line to the reactor

The FORECON Sniper:
Currently, FORECON GL is kinda unfitting and also kinda sucks the sniper, on the other hand, would be a perfect fit for FORECON now I'm not going to pretend the sniper isn't kinda OP right now so the FORECON sniper won't actually get his M42A just the ghillie suit that gives him still some uniqueness as a survivor as well as an incentive to survive beyond the life of the regular sniper (to steal his gun)

Testing Photographs and Procedure

Screenshots & Videos

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

Changelog

🆑 SpartanBobby
add: Added a new survivor to LV522 the FORECON sniper he spawns alone his only company being the corpse of his dead battle buddy, good luck
maptweak: Sweeping changes to LV522 including the reactor, north of engineering, LZ1, the entire central area of the map, and north of fitness in an attempt to see more of the colony used and to incentivize flanking
/:cl:

@github-actions
Copy link
Contributor

You currently have a negative Fix/Feature pull request delta of -45. Maintainers may close this PR at will. Fixing issues or improving the codebase will improve this score.

@github-actions github-actions bot added Mapping did you remember to save in tgm format? Feature Feature coder badge labels Jul 18, 2023
maps/lv522_chances_claim.json Show resolved Hide resolved
@Nanu308 Nanu308 marked this pull request as draft July 22, 2023 03:55
Copy link
Member

@Nanu308 Nanu308 left a comment

Choose a reason for hiding this comment

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

e

@spartanbobby spartanbobby marked this pull request as ready for review July 22, 2023 10:15
@realforest2001 realforest2001 changed the title (Requesting tactical testmerge) Sweeping LV522 Chances + FORECON GL being replaced by FORECON Sniper (Without the sniper) Sweeping LV522 Changes + FORECON GL replaced by FORECON Sniper (Without the sniper) Jul 24, 2023
@Nanu308 Nanu308 added the Mapping Approved adds 500 new dict keys label Jul 27, 2023
@stagda stagda mentioned this pull request Jul 28, 2023
3 tasks
@harryob harryob enabled auto-merge July 28, 2023 06:39
@harryob harryob added this pull request to the merge queue Jul 28, 2023
Merged via the queue into cmss13-devs:master with commit 9c79c3a Jul 28, 2023
25 checks passed
cm13-github added a commit that referenced this pull request Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge Mapping Approved adds 500 new dict keys Mapping did you remember to save in tgm format? Testmerge Candidate we'll test this while you're asleep and the server has 10 players
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants