Skip to content

Reduce sniper speed and increase range - #7167

Merged
Rhaelya merged 5 commits into
developfrom
balance/sniper-speed
Jul 12, 2026
Merged

Reduce sniper speed and increase range#7167
Rhaelya merged 5 commits into
developfrom
balance/sniper-speed

Conversation

@lL1l1

@lL1l1 lL1l1 commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Description of the proposed changes

  • Reduce sniper bot speed so that they can't kite GC/Ythotha/Monkeylord, as otherwise they have no lategame counter for Aeon/Sera. This also makes them weaker against T3 since its harder to escape a micro mistake.
  • Add 5 range to compensate the speed nerf.

Some considerations of the effects of these changes:

  • Reaction time for starting kiting is higher with more range, but kiting in general is worse with less speed.
  • More units/HP is needed to walk into range of an idle sniper army due to higher range.
  • Ythotha and GC had a somewhat small range gap previously, so +5 range increase to that small gap is a big difference, especially for Seraphim sniper in normal mode. Ythotha 47 range vs 55/60/65 = 8/13/18 gap -> 13 (+62.5%)/18/23 gap. GC 40 range so 15/20/25 gap -> 20/25/30 gap
  • Megalith gets outranged by 1 -> 6 by Seraphim snipers in sniper mode, but Seraphim snipers are 0.3 slower so mega catches up in only 20s, which means that the snipers can get only 2 shots off before they start getting shot at with high range high damage splash.

Testing done on the proposed changes

To be tested on develop/beta.

Checklist

Summary by CodeRabbit

  • Balance
    • Reduced mobility for T3 sniper bots by lowering movement/tuning speeds (including standard and sniper-mode movement).
    • Increased engagement range to compensate, expanding weapon effectiveness for both standard and sniper firing modes across the affected variants.
  • Chores
    • Removed leftover placeholder content from an empty changelog snippet entry.

lL1l1 added 2 commits July 5, 2026 14:31
fixes them permanently kiting T4s
round sera snipe mode speed from 1.74 to 1.7 as a bugfix of the speed reduction mult applying twice
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

This PR reduces movement speeds for XAL0305 and XSL0305, adjusts XSL0305 sniper-mode movement, and increases weapon ranges. It adds a balance changelog entry and removes a stale placeholder snippet.

Changes

T3 Sniper Bot Balance Adjustment

Layer / File(s) Summary
XAL0305 movement and weapon range tuning
units/XAL0305/XAL0305_unit.bp
MaxAcceleration, MaxBrake, and MaxSpeed are reduced from 2.5 to 2.3; main weapon MaxRadius increases from 60 to 65.
XSL0305 movement and weapon range tuning
units/XSL0305/XSL0305_unit.bp
Physics values are reduced from 2.3 to 2.2, SniperModeSpeedMultiplier changes to 0.879, and weapon ranges increase to 60 and 70.
Changelog documentation update
changelog/snippets/balance.7167.md, changelog/snippets/category.7158.md
Adds the balance entry and removes the stale placeholder snippet.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • FAForever/fa#7142: Modifies the same T3 sniper bot blueprints and tuning values.
  • FAForever/fa#7158: Edits the same sniper-bot blueprint parameters with opposite tuning changes.

Suggested reviewers: Rhaelya, BlackYps

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: reducing sniper speed and increasing range.
Description check ✅ Passed The description matches the template well, with proposed changes, testing, and checklist covered.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch balance/sniper-speed

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 added the area: balance idea related to suggestions for unit balance label Jul 6, 2026
@lL1l1
lL1l1 requested a review from Rhaelya July 6, 2026 05:41
@lL1l1 lL1l1 changed the title Balance/sniper speed Reduce sniper speed and increase range Jul 6, 2026
@lL1l1
lL1l1 marked this pull request as ready for review July 6, 2026 05:41
Fix wrong speed in snippet for Seraphim sniper

@Rhaelya Rhaelya left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@changelog/snippets/balance.7167.md`:
- Line 12: Update the Snipe mode speed entry in the changelog to document the
corrected Usha-Ah sniper-mode value of approximately 1.93, replacing the
inaccurate 1.7 value while preserving the existing before-value and formatting.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8ad48d7e-add3-43b0-84a0-a64cf9b1de15

📥 Commits

Reviewing files that changed from the base of the PR and between 6be5a45 and 28890ff.

📒 Files selected for processing (1)
  • changelog/snippets/balance.7167.md

- Range: 60 -> 65
- Usha-Ah: T3 Sniper Bot
- Max speed: 2.3 -> 2.2
- Snipe mode speed: 1.74 -> 1.7

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Correct the documented Usha-Ah sniper-mode speed.

The entry says 1.74 -> 1.7, but the supplied XSL0305 configuration uses MaxSpeed = 2.2 and SniperModeSpeedMultiplier = 0.879, yielding approximately 1.93. Update this value to match the actual blueprint change.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@changelog/snippets/balance.7167.md` at line 12, Update the Snipe mode speed
entry in the changelog to document the corrected Usha-Ah sniper-mode value of
approximately 1.93, replacing the inaccurate 1.7 value while preserving the
existing before-value and formatting.

@Rhaelya
Rhaelya merged commit 64f7588 into develop Jul 12, 2026
5 checks passed
@Rhaelya
Rhaelya deleted the balance/sniper-speed branch July 12, 2026 16:09
@github-project-automation github-project-automation Bot moved this from To Discuss to Done in 2025 - Balance Iteration I Jul 12, 2026
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: Done

Development

Successfully merging this pull request may close these issues.

3 participants