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

HvH faction remastering and adding: Vyacheslav Security Detail pt.1 #15844

Merged
merged 28 commits into from May 25, 2024

Conversation

SomrNetwork
Copy link
Contributor

@SomrNetwork SomrNetwork commented May 11, 2024

About The Pull Request

With the climb of Campaign, I've gotten an inspiration to make and remaster some factions. Starting with my first, the Vyacheslav Security Detail. A Syndicate PMC squad sent to assist in ICC and SoM operations. Equipped with Crasher ballistic armor and Cold War weaponries. Consisting of the following:

The C550 Assault Rifle, chambered in 10x27mm
image

The L26 Support MG, chambered in 5.56mm
image

The C17 PDW, firing AP .45 ACP
image

The L12 pump-action Shotgun, 12 gauge
image

The XM104 'Divider' Revolver, firing incendiary .357
image

The C96 pistol, chambered in 9mm parabellum
image

The C153 shoulder launcher, fires HE and Chemical warheads Incendiary rounds
image

and the armory:

image

image

image

image

extras:
image

image

image

Why It's Good For The Game

Campaign seems to lack a few faction varieties. I want to assist in adding a slight amount of variation with lore, factions, guns and armor. And, since TGMC has slightly more factions than ICC/SoM, I wish to balance it too.

Changelog

🆑
add: vsd faction
add: New magfed revolvers!
imageadd: campaign portrait and UI
imageadd: VSD armor and weapons
balance: faction balance
/:cl:

vee ess dee
@tgstation-server tgstation-server added Sprites Changes to .dmi file. Merge Conflict Pull request is in a conflicted state with base branch. Balance Changes to functionality that modifies how effective certain methods are at powergaming. Feature New interesting mechanics with new interesting bugs labels May 11, 2024
@SomrNetwork
Copy link
Contributor Author

time to update the branch to current TGMC!!

@tgstation-server tgstation-server removed the Merge Conflict Pull request is in a conflicted state with base branch. label May 11, 2024
@Tailiik
Copy link

Tailiik commented May 11, 2024

Looks good, you might wanna put a comment on the crasher armor declaration that it is used for your faction though so it doesn't get snapped when the mk3ening happens.

@SomrNetwork
Copy link
Contributor Author

kuro doesn't touch much of the faction armors. I don't really think he'd touch the ERT crasher armor anyway

on another note
WHAT THE FUCK
image

@lbnesquik
Copy link
Contributor

"We need bad guys for our f-"

The Russian PMC :

ronaldo-win-ronaldo-award-ezgif com-resize

@Lumipharon
Copy link
Contributor

bro how old is your branch, and did you just ignore all the conflicts when you pulled master?

@SomrNetwork
Copy link
Contributor Author

bro how old is your branch, and did you just ignore all the conflicts when you pulled master?

this project started 3 months ago so

@Tonklaatech
Copy link

Z TARDS RISE UP!!!

Copy link
Contributor

@Lumipharon Lumipharon left a comment

Choose a reason for hiding this comment

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

My eyes are glazing over from the volume of gear you're adding, but here's a couple of things to fix

Comment on lines 91 to 94
#define FREQ_VSD 1351
#define MAX_ERT_FREQ 1351

#define FREQ_AI 1351
Copy link
Contributor

Choose a reason for hiding this comment

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

hrm.

@@ -567,3 +567,136 @@
/obj/item/clothing/glasses/night_vision/mounted/Initialize(mapload)
. = ..()
ADD_TRAIT(src, TRAIT_NODROP, NIGHT_VISION_GOGGLES_TRAIT)

/obj/item/clothing/glasses/vsd/nvg
Copy link
Contributor

Choose a reason for hiding this comment

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

did you just copy paste the entire code for existing nvg's?

nvg's are now on nvg code, and freq fixed. Also fixes the gun on mobs
@SomrNetwork
Copy link
Contributor Author

now waiting on kuro's sprite approvals

@Lumipharon
Copy link
Contributor

yeah just an FYI I wouldn't even want to look at walance until sprites are reviewed.

@SomrNetwork
Copy link
Contributor Author

its been 3 days since Kuro's approval

code/game/objects/effects/effect_system/smoke.dm Outdated Show resolved Hide resolved
code/modules/clothing/glasses/glasses.dm Outdated Show resolved Hide resolved
code/modules/clothing/head/helmet.dm Outdated Show resolved Hide resolved
code/modules/projectiles/guns/rifles.dm Outdated Show resolved Hide resolved
code/modules/projectiles/guns/shotguns.dm Outdated Show resolved Hide resolved
code/modules/projectiles/guns/smgs.dm Outdated Show resolved Hide resolved
code/modules/projectiles/guns/specialist.dm Outdated Show resolved Hide resolved
code/datums/gamemodes/campaign/rewards/allies.dm Outdated Show resolved Hide resolved
@Lumipharon
Copy link
Contributor

wotdis sprite
image

its like not even the same palette as the other guns, and is just really low quality.

@SomrNetwork
Copy link
Contributor Author

time to annoy Lumi again!!

@@ -743,22 +743,22 @@ Fight for TGMC, and attempt to achieve all objectives given to you."})
job_flags = JOB_FLAG_LATEJOINABLE|JOB_FLAG_BOLD_NAME_ON_SELECTION|JOB_FLAG_PROVIDES_SQUAD_HUD|JOB_FLAG_CAN_SEE_ORDERS
job_cost = 0
outfits = list(
/datum/outfit/job/vsd/standard/c550/campaign,
/datum/outfit/job/vsd/standard/gruntone/campaign,
Copy link
Contributor

Choose a reason for hiding this comment

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

camel_case. /grunt_one. or just /one

deactive_state = "vsd_nvg_off"
toggleable = TRUE
tint = COLOR_GREEN
item_flags = DELONDROP
Copy link
Contributor

Choose a reason for hiding this comment

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

whydis?

/obj/item/weapon/gun/flamer/mini_flamer,
/obj/item/weapon/gun/grenade_launcher/underslung,
/obj/item/weapon/gun/rifle/pepperball/pepperball_mini,
/obj/item/weapon/gun/grenade_launcher/underslung/mpi,
Copy link
Contributor

Choose a reason for hiding this comment

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

I was only questioning the mpi UGL btw, since thats just a normal UGL but with a custom sprite specifically to fit the MPI, and it is unremovable.

tgmc.dme Outdated Show resolved Hide resolved
tgmc.dme Outdated Show resolved Hide resolved
@Lumipharon Lumipharon added the Test Merge Candidate This PR has been reviewed and is ready for testing, unless something changes. label May 21, 2024
@tgstation-server tgstation-server removed the Test Merge Candidate This PR has been reviewed and is ready for testing, unless something changes. label May 22, 2024
@SomrNetwork
Copy link
Contributor Author

What the fuck tgstation bot

comfyorange added a commit that referenced this pull request May 23, 2024
comfyorange added a commit that referenced this pull request May 23, 2024
@TiviPlus TiviPlus merged commit 364f0db into tgstation:master May 25, 2024
34 checks passed
github-actions bot added a commit that referenced this pull request May 25, 2024
@SomrNetwork
Copy link
Contributor Author

i'm pretty sure this is still testmerged but yipee!!

@Lumipharon
Copy link
Contributor

I wanted to tm for walance but ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Changes to functionality that modifies how effective certain methods are at powergaming. Feature New interesting mechanics with new interesting bugs Sprites Changes to .dmi file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants