forked from sojourn-13/sojourn-station
-
Notifications
You must be signed in to change notification settings - Fork 18
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Reverts Sojourn's PR 5029 armour changes (#69)
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> ## About The Pull Request <details> <summary> This PR aims to revert armour changes that happened in sojourn-13#5029 </summary> <hr> This draft is here mostly to make it visible (not just a branch on my fork), not sure why I have done it honestly but I was bored, I done it, and now it is here. <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> <hr> </details> This draft does the following : - Reverts the armour code to pre 5029 (Bulk combat changes; Armor Divisors, bullet balance, mob health/armor, etc.) PR state - Keeps most of other changes that the PR made (like global health modifiers for PVE factions or blocking) - Adds ABSOLUTE armour to the armour calculation (currently disabled, controlled by ABSOLUTE_ARMOR_EFFICIENCY variable) - Adds RELATIVE_ARMOR_EFFICIENCY and ABSOLUTE_ARMOR_EFFICIENCY both being multipliers for their respective armour type What is not yet there : - Overpenetration/halloss damage in PVE - ablative_armor is not accounted for in armour calculations - ALOT OF TESTING - ALOT OF CLEANUP (Both armor_penetration and armor_list should have their values set by defines among other things) ### SO WHY DID I DO IT I don't know, but if I did it already might as well keep it around as a draft ? perhaps it will be educational, or something Technically we could merge it (after fixes) but as far as I understand we are still testing and figuring out current armour system. There probably are broken things, it feels like I frankensteined alot of things together into a lump that works. code\modules\mob\living\living_defense.dm above all will need a look at, alot of commented out code for PVE interactions especially ## Changelog :cl: add: Added ABSOLUTE and RELATIVE armour to armour system del: Removed old code for armour tweak: tweaked armour balance: rebalanced armour back to what it was (more or less) /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> --------- Co-authored-by: Takykardi <[email protected]> Co-authored-by: valkyria-gk <[email protected]>
- Loading branch information
1 parent
87370aa
commit 4c203e7
Showing
230 changed files
with
1,756 additions
and
1,700 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.