Compat SPE / SPEX - Add ace_overheating_barrelMass to small arms#11413
Conversation
|
There is no need to add this if you're just making it 55% of the weapon's weight. 55% is already the default fallback value if there is no barrelMass config entry. |
Hi Drofseh, I did some very light research this evening and scrounged the various forums and gun brokerages for reproductions in similar calibers or original models. Where I could not find a barrel I used weapons in the same category or caliber. I updated my original post with links and some of my methodology. I hope this makes the PR more acceptable / valuable for the players. |
johnb432
left a comment
There was a problem hiding this comment.
Thanks for the PR.
There are two things I'd like to ask:
- You sometimes specify sources, sometimes there are none. Why is that? It's best to be consistent.
Could be similar to x maybe ycomments are in my opinion not a good descriptor.couldis too vague; either remove the comment or update it to something similar toEstimated slightly y to x
Co-authored-by: johnb432 <58661205+johnb432@users.noreply.github.com>
Co-authored-by: johnb432 <58661205+johnb432@users.noreply.github.com>
Some weapons I just couldn't find a source for the actual barrel mass sources so I used a i.e a STEN Mk 2 and a MP40 are both sub machineguns with similar weights in real life I updated the comments to use the word 'estimated' because like above I just estimated or grouped similar weapon categories / weights together. I'd like to keep it there in case someone in the future who looks wants to see where I sourced my information from. |
ace_overheating_barrelMass to small arms
When merged this pull request will:
Barrel weight sources off of the internet
IMPORTANT
Component - Add|Fix|Improve|Change|Make|Remove {changes}.