toadie2k / NIArms

45 stars 44 forks source link

Small Mass, Ballistics, Naming, Recoil Fixes #172

Closed LinkIsGrim closed 2 years ago

LinkIsGrim commented 4 years ago

When merged, this PR will:

Blutze commented 4 years ago
Apply proper name string to hlc_lmg_minimipara_long_railed
Correct ballistics for HLC_300Blackout_SMK

These were already included in #156 and #168. In my experience, git diff is unreliable when it comes to xml files, so this is a case for caution when merging. Also, initSpeed (which is usually intended to equal typicalSpeed) on the TAC-TX mag is 670.6, not 601.

Not sure I agree with some of the other balancing changes here, though that is from a pure balancing and game design POV, not realism.

LinkIsGrim commented 4 years ago

Missed that on the TAC-TX, fixing to match initSpeed. ACE doesn't typically care for it, but I forget most play without it.

As for balancing, if you're referring to the recoils, I followed the example set in hlc_wp_ar15, which uses recoil_spar for majority of the weapons, and applied the existing BI recoils to the classes. hlc_wp_ak doesn't seem to have been overhauled since the release of the APEX DLC, which was when such recoils were added. hlc_aek_971 was changed to recoil_aks because I feel it doesn't make much sense for a 5.45 AK to have the same recoil as 7.62, particularly when they're differentiated in the same config.