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

Balancing vehicle skills across USCM #7798

Closed
wants to merge 1 commit into from

Conversation

majchroo
Copy link
Contributor

@majchroo majchroo commented Dec 12, 2024

About the pull request

tl;dr; Add VAN driving skills to many USCM roles. Add ARC skills to CE

Current state of VAN skills:

General, CO, XO, ASO, SO, SL, Combat Engineer

New state of VAN skills:

General, CO, XO, ASO, SO, SL, Combat Engineer, Spec FTL, IO, DP, DCC, MP, MW, MT, CT, CMP, RO

Current state of ARC skills:

General, CO, XO, SO, SEA

New state of ARC skills:

General, CO, XO, SO, SEA, CE

Explain why it's good for the game

IO's - It's logical for mobile officers working alone to have driving skills
CE+MT - they can fix em but can't ride em as of now
Spec, FTL - because they often go aSL and have wide range of skills
DP, DCC - this is basically to allow them to move small vehicles out of their dropships
MP,MW,CMP - military police cant handle small transport vehicles as of now
RO, CT - logistics cant handle small small transport vehicles as of now

Further forum discussion: https://forum.cm-ss13.com/t/balancing-vehicle-skills-across-uscm/11688/15

Testing Photographs and Procedure

Spawned Spec and CT. Can confirm that they can drive VAN after the change.

Screenshots & Videos

Screenshot

Changelog

🆑Adamix147
balance: Grant VAN vehicle skills across USCM. Grant ARC skills to CE.

/:cl:

@cmss13-ci cmss13-ci bot added the Balance You need to be a professional veteran game maintainer to comprehend what is being done here. label Dec 12, 2024
@majchroo majchroo marked this pull request as ready for review December 12, 2024 17:04
@Red-byte3D
Copy link
Member

#5401 pretty sure @Zonespace27 was against giving anything but actual CIC command training for the arc

@Somniworld
Copy link

#5401 pretty sure @Zonespace27 was against giving anything but actual CIC command training for the arc

Hiya, is that why IOs can't drive the ARC? I understood that it was meant to serve as a drop-off point for the intel team, which is currently very difficult to do when only SOs reliably ever drive them.

Perhaps replacing the CE with IOs would be more thematic ? Especially since they get van driving skills in this very PR.

@Zonespace27
Copy link
Contributor

Not entirely sold on the benefit of having CEs be able to drive the ARC. (Having IOs do it would be worse so don't do that)

@majchroo
Copy link
Contributor Author

Hiya, is that why IOs can't drive the ARC? I understood that it was meant to serve as a drop-off point for the intel team, which is currently very difficult to do when only SOs reliably ever drive them.

@Somniworld I found this while working on this PR:
https://forum.cm-ss13.com/t/give-ios-the-ability-to-drive-the-arc/9295

Not entirely sold on the benefit of having CEs be able to drive the ARC. (Having IOs do it would be worse so don't do that)

@Zonespace27 Let me know if you want it removed from the PR.

I wont push it because it serves no other gameplay purpose than reparking it in FOB. CE can't really leave FOB/comms because of SOP anyway.

I did add it because:

  1. Reparking
  2. It was voiced on forum that it's logical for master of engineering to have driving skills.

@HeresKozmos
Copy link
Contributor

IOs shouldn't have driving skill, them picking up a vehicle and driving around the colony negates the biggest threat to them which is them having to travel alone/in small groups into the sideline/backline, and it will also let them just fill the vehicle with giant piles of intel basically negating any reason to travel back and forth unless they just stock up giant piles of it.

@DorfGuy1
Copy link

Add it to medics too. Marine ambulance. Ugh. I've been saying for months that marines should get a truck by default as a correspondence to the mobility that tunnels offer.

@Ammoniacres
Copy link

IOs shouldn't have driving skill, them picking up a vehicle and driving around the colony negates the biggest threat to them which is them having to travel alone/in small groups into the sideline/backline, and it will also let them just fill the vehicle with giant piles of intel basically negating any reason to travel back and forth unless they just stock up giant piles of it.

IOs having driving skills would be incredibly fun, but negating the danger of travelling should come with some sincere risks or a cost beyond finding wheels. It also shouldn't be great for carting intel.

@cmss13-ci
Copy link
Contributor

cmss13-ci bot commented Jan 23, 2025

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@cmss13-ci cmss13-ci bot added the Stale beg a maintainer to review your PR label Jan 23, 2025
@cmss13-ci cmss13-ci bot closed this Feb 11, 2025
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. Stale beg a maintainer to review your PR
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

7 participants