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

Buff autoloaders #377

Merged
merged 3 commits into from
Nov 27, 2023
Merged

Conversation

thecraftianman
Copy link
Member

Autoloaders are easily one of the worst guns at the moment. This should make them significantly more viable in comparison to other options (but hopefully not too overpowered; if this seems too strong I'll fix it lol):

  • Decreased mass by 47%
  • Increased mag size by 2
  • Increased overall RPM by 3

@thecraftianman thecraftianman changed the base branch from master to dev October 26, 2023 15:57
@TwistedTail
Copy link
Member

I'm not so sure about the mass buff, I remember changing it on all guns so it was somehow consistent with the model and the density of steel. I think we could honestly review that as a whole and use a better method to get the weapon's volume, since it's using the entity's volume and it's not really correct to assume we're dealing with a solid chunk of steel.

@thecraftianman
Copy link
Member Author

Imo the mass buff here is probably the most important part. Autoloaders weigh many times more than any of the other guns as they are now; for the mass that a single 170mm autoloader takes up, you could instead have 4 203mm howitzers and still have a couple tons to spare. Even if all weights are based off the model size and the density of steel, their weight still seems inconsistent, since a 170mm autoloader would weigh ~17 tons more than a 170mm cannon, which is relatively comparable in model size.

Either the autoloader physobj needs to be updated or a new, similarly consistent method of determining weapon mass needs to be worked out; see https://discord.com/channels/654142834030542878/874391629891526787/1167548702286675998
@thecraftianman thecraftianman merged commit 5e18d3d into ACF-Team:dev Nov 27, 2023
2 checks passed
@thecraftianman thecraftianman deleted the autoloader-buff branch November 27, 2023 18:12
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