Skip to content

partial sacu rebalance iteration 2#7164

Open
lL1l1 wants to merge 12 commits into
developfrom
balance/partial-sacu-rebalance-iteration-2
Open

partial sacu rebalance iteration 2#7164
lL1l1 wants to merge 12 commits into
developfrom
balance/partial-sacu-rebalance-iteration-2

Conversation

@lL1l1

@lL1l1 lL1l1 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description of the proposed changes

Applying a variety of changes to the partial sacu rebalance (#7141) based on feedback and experience of fafdevelop games.
Specific changes and their explanations can be found in the commit history.

Testing done on the proposed changes

To be tested in fafdevelop/fafbeta.

Checklist

- [ ] Changes are annotated, including comments where useful

Summary by CodeRabbit

  • Balance Changes
    • SACU build speed and upgrade costs have been rebalanced across multiple factions.
    • Several support and combat upgrades now cost more or build more slowly, especially stronger options.
    • Some shield and repair-related upgrades have been adjusted for better overall balance.
    • SACUs no longer prioritize enemy command units, making overcharge counterplay more reliable.
    • Seraphim SACU loadouts and shield placement have been updated, including a revised preset option.

lL1l1 added 11 commits July 3, 2026 01:05
1k is too low because it makes the early SACU too strong, but reverting to 4.5k makes later SACUs too weak.
E/M of adv combatant: 42.6 (v3836) -> 35 (iter 1) -> 38 (iter 2)
BT adjusted to keep adv combatant at 34m/s from gateway
Its compounding effects of hp and regen make it a bit too strong compared to other factions
OC E cost reduced to keep adv. combatant 38 e/m
ACU is a key counter to SACU rushes
There were some concerns that the upgrade was too weak after the rework from stats-analyzing players. In test games I thought it was ok although it recharged a bit too slowly. After comparing the recharge hp/s with mobile shields it did seem in need of a small buff.
Combatant: 25.6 (v3836) -> 24.5 (rebal 1) -> 26.5 (rebal 2)
Rambo: 34 -> 32 -> 35
combatant 25 to 28
its emp so its quite strong vs t3 so it shouldnt be as early as say uef combatant

Rambo 34 to 35
matches uef rambo
snowbally big range big aoe unit even stronger than cyb combatants
SACU bp in general is very strong but its hard to decrease it due to them being so much more advanced and larger than T3 engis, so it is set to just 42 as a bit of an experimental venture to see how players respond (sacu mod was 25). Reduced bp also slightly justifies reduced cost of all presets.
UEF has a weak pushing t4, ravagers so it would be exemplify the faction traits for them to keep their bp. They also have the most expensive SACU and the weakest engi preset (less total bp + drone that dies to aa in combat).
@lL1l1 lL1l1 requested a review from Rhaelya July 3, 2026 08:35
@lL1l1 lL1l1 added the area: balance idea related to suggestions for unit balance label Jul 3, 2026
@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6ee55166-995f-429a-ac8c-a1037f8ae77f

📥 Commits

Reviewing files that changed from the base of the PR and between 7b9f49c and 96bfa33.

📒 Files selected for processing (6)
  • changelog/snippets/balance.7141.md
  • loc/US/strings_db.lua
  • units/UAL0301/UAL0301_unit.bp
  • units/UEL0301/UEL0301_unit.bp
  • units/URL0301/URL0301_unit.bp
  • units/XSL0301/XSL0301_unit.bp

📝 Walkthrough

Walkthrough

This PR rebalances Support Armored Command Units (SACUs) across all four factions (UEF, Aeon, Cybran, Seraphim). Changes include removed SNIPEMODE categories, removed COMMAND target priorities, adjusted BuildRate/buildpower values, revised enhancement build costs, shield/upgrade retuning, updated changelog documentation, and localized "+56 Buildpower" description strings.

Changes

SACU Rebalance

Layer / File(s) Summary
Changelog rebalance notes
changelog/snippets/balance.7141.md
Documents SACU-wide rebalance theme, Seraphim shield/RAS changes, removal of "prioritize ACUs" ability, and per-faction cost/stat adjustments (Quantum Gateways, UEF bubble shield, engineering upgrades, Cybran/Seraphim upgrades).
Localized buildpower string updates
loc/US/strings_db.lua
Updates Cybran, Aeon, and Seraphim SACU build-speed description strings from +42 to +56 Buildpower.
Aeon UAL0301 blueprint changes
units/UAL0301/UAL0301_unit.bp
Removes SNIPEMODE category and COMMAND target priority, lowers BuildRate from 56 to 42, and raises Shield and SystemIntegrityCompensator enhancement energy costs.
UEF UEL0301 blueprint changes
units/UEL0301/UEL0301_unit.bp
Removes SNIPEMODE category and COMMAND target priority, raises AdvancedCoolingUpgrade/HighExplosiveOrdnance/Shield enhancement energy costs, and reduces ShieldGeneratorField recharge time.
Cybran URL0301 blueprint changes
units/URL0301/URL0301_unit.bp
Removes SNIPEMODE category and COMMAND target priority, lowers BuildRate from 56 to 42, and raises EMPCharge, FocusConvertor, and SelfRepairSystem enhancement energy costs.
Seraphim XSL0301 blueprint changes
units/XSL0301/XSL0301_unit.bp
Removes SNIPEMODE category and COMMAND target priority across three weapon blocks, lowers BuildRate from 56 to 42, and raises DamageStabilization and Overcharge enhancement cost/build time.

Estimated code review effort: 2 (Simple) | ~15 minutes

Possibly related PRs

  • FAForever/fa#7142: Removes "SNIPEMODE" category and "COMMAND" from TargetPriorities across similar unit blueprints.
  • FAForever/fa#7158: Also modifies "SNIPEMODE"/"COMMAND" target priority entries in unit blueprint files.

Suggested labels: area: balance

Suggested reviewers: Rhaelya

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the PR's main focus on a second SACU rebalance iteration.
Description check ✅ Passed The description includes the required sections and is mostly complete, with a clear summary, testing note, and checklist.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch balance/partial-sacu-rebalance-iteration-2

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@lL1l1 lL1l1 marked this pull request as ready for review July 3, 2026 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: balance idea related to suggestions for unit balance

Projects

Status: To Discuss

Development

Successfully merging this pull request may close these issues.

1 participant