Skip to content

Change: China Tank Battlemaster with Autoloader will always reload when idle #1704

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

Merged
merged 1 commit into from
Sep 17, 2023

Conversation

xezon
Copy link
Contributor

@xezon xezon commented Feb 13, 2023

With this change the China Tank Battlemaster with Autoloader will always reload when idle.

This makes the unit significantly better in scenarios where it managed to only fire 1 or 2 times and some time passes until the next target is attacked.

This situation is common. There is a ClipReloadTime of 2000 ms vs a DelayBetweenShots of 533 (500) ms with 3 clips. This means there is a 44% chance that Tank Battlemaster with Autoloader gets into state with just 1 or 2 loaded clips in regular battle engagements.

Rationale

Eliminates situations where Tank Battlemaster with Autoloader would only fire 1 or 2 shells on a fresh engagement. This likely is what player would expect always.

Original

generals.2023-02-13.20-20-55-82.mp4

Patched

generals.2023-02-13.20-28-51-03.mp4

@xezon xezon added Design Is a matter of game design Controversial Is controversial Major Severity: Minor < Major < Critical < Blocker Buff Makes a thing more powerful China Affects China faction labels Feb 13, 2023
@xezon
Copy link
Contributor Author

xezon commented Feb 13, 2023

I am skeptical of this change. Makes this unit much stronger.

@commy2
Copy link
Collaborator

commy2 commented Feb 13, 2023

I don't think it makes much of a difference, but would be neat to see a reloaded and a random group with same veterancy duke it out.

@penfriendz
Copy link

I'd fix it then tweak if it's op. Consistent expected behaviour is good.

@Stubbjax
Copy link
Collaborator

Stubbjax commented Feb 14, 2023

I wouldn't say this change makes it much stronger. The Autoloader upgrade is rather misleading in that it only increases damage per second by ~30%, when it should be expected to be closer to 200% due to the additional two shells. Nationalism can be considered marginally better in 1.04 (due to #810 / TheSuperHackers/GeneralsGameCode#173) at +25% DPS for $500 less.

BattleMasterTankGun deals 1×60 damage per 60 frames and Tank_BattleMasterTankGun deals 3×40 (120) damage per 92 (16 + 16 + 60) frames. Note that the 500ms delay between each shot in the clip translates to 533.33ms and thus 16 frames.

Upgrade Damage Fire Rate DPS
None 60 2,000ms 30
Nationalism 60 1,600ms 37.5
Autoloader 120 3,067ms 39.13
Both 120 2,400ms 50

It's also important to note that Tank Battlemasters start at Veteran rank, and often Elite rank. This reduces the window for a partial clip to less than 24 frames (2×400ms) and 20 frames (2×333ms), respectively. The remaining shell(s) may also still be fired before the clip's reload duration. Movement may also significantly extend these windows.

So if an Autoloader Battlemaster only fires one shot, it is technically worse than without the upgrade. If it fire two shots, it is only marginally better. I don't see anything overpowered about this.

@MTKing4
Copy link
Collaborator

MTKing4 commented Feb 14, 2023

Looks good

@xezon
Copy link
Contributor Author

xezon commented Feb 14, 2023

Would someone be able to make test map with classic Tank Gun vs patched Tank Gun in big battle to see difference, if there is any?

@kABUSE1
Copy link
Collaborator

kABUSE1 commented Apr 13, 2023

Map to test battlemaster strength attached.

Let me know what you think or if adaptions are required.

Play this map vs. tank general and just let it run.

Auto-loader will be purchased automatically.

After 60 sec quads keep spawning in. Amount of kills is being tracked and can be seen in bottom right corner.

battlemaster-strength-tester.zip

@xezon
Copy link
Contributor Author

xezon commented Apr 13, 2023

I ran test 10 times on test map vs Tank General, Medium Army.

Before this change

Traitors Killed: 63
Traitors Killed: 59
Traitors Killed: 59
Traitors Killed: 59
Traitors Killed: 59
Traitors Killed: 59
Traitors Killed: 59
Traitors Killed: 59
Traitors Killed: 59
Traitors Killed: 59

After this change

Traitors Killed: 46
Traitors Killed: 49
Traitors Killed: 49
Traitors Killed: 49
Traitors Killed: 49
Traitors Killed: 49
Traitors Killed: 49
Traitors Killed: 49
Traitors Killed: 49
Traitors Killed: 49

Conclusion

???

Judging by these numbers it implies that the Battlemaster now performs worse than before. How is this possible? It is supposed to perform better.

@xezon
Copy link
Contributor Author

xezon commented Apr 13, 2023

I ran test anew vs Tank General, Medium Army. But this time I did not Restart map with button, and instead went back to lobby and started new game. I think only this will reset the random seed to new value. Result numbers ordered.

Before this change:

35, 59, 60, 76, 78, 97, 103, 117, 118, 253

Average: 99.6

After this change:

49, 68, 79, 96, 113, 121, 126, 157, 202, 251

Average: 126.2

Conclusion

There is too much variation in this test. 35 to 253 is just too much discrepancy. We would need to collect thousands of samples to get a meaningful average. The lower the sample count, the more randomness it will contain.

@xezon xezon force-pushed the change-battlemaster-autoloader-reload branch from 5db0d20 to 5478c69 Compare April 15, 2023 07:16
@xezon
Copy link
Contributor Author

xezon commented Jun 3, 2023

It looks like we cannot evaluate impact of this change in a meaningful way. So we would have to yolo it and see how it turns out.

@xezon xezon force-pushed the change-battlemaster-autoloader-reload branch from 5478c69 to 9fc4fe9 Compare September 17, 2023 15:44
@xezon xezon force-pushed the change-battlemaster-autoloader-reload branch from 9fc4fe9 to 59a637b Compare September 17, 2023 15:50
@xezon
Copy link
Contributor Author

xezon commented Sep 17, 2023

Documentation added.

If China Tank becomes too strong, we have many thing to dial back.

@xezon xezon merged commit 1b1fca1 into main Sep 17, 2023
@xezon xezon deleted the change-battlemaster-autoloader-reload branch September 17, 2023 15:51
@ItsWazzaP ItsWazzaP added the GameMismatchWith104 Affects game state towards original 1.04 label Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Buff Makes a thing more powerful China Affects China faction Controversial Is controversial Design Is a matter of game design GameMismatchWith104 Affects game state towards original 1.04 Major Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

China Tank Battlemaster with Autoloader does not auto reload before entire clip is emptied
7 participants