Skip to content

[C++] Update Automaton Base Damage Calculation#10319

Open
Skold177 wants to merge 1 commit into
LandSandBoat:basefrom
Skold177:Update-Automaton-Base-Damage-Calculation
Open

[C++] Update Automaton Base Damage Calculation#10319
Skold177 wants to merge 1 commit into
LandSandBoat:basefrom
Skold177:Update-Automaton-Base-Damage-Calculation

Conversation

@Skold177

@Skold177 Skold177 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

  • Improves the accuracy of the automaton base damage calculation. This is a slight buff over the old one and results in roughly 3 more weapon damage at 99. (From 97 at max skill to 100)
image

Steps to test these changes

Check your automaton with !getstats 3, see very slightly higher weapon damage.

Capture Log

Veralice.log

This was analyzed by Jimmayus who concluded my automaton had 100 base damage, which was 3 higher than my LSB local.

@WinterSolstice8 WinterSolstice8 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.

CI will probably complain about these on clang tidy

Comment thread src/map/utils/petutils.cpp Outdated
Comment thread src/map/utils/petutils.cpp Outdated
Improves the calculation for automaton base damage with a capture from retail and analysis from Jimmayus
@Skold177 Skold177 force-pushed the Update-Automaton-Base-Damage-Calculation branch from 6eb76f8 to 75532fc Compare June 15, 2026 02:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants