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

Increate Plate armor cover to 98% #49338

Closed
wants to merge 1 commit into from
Closed

Increate Plate armor cover to 98% #49338

wants to merge 1 commit into from

Conversation

Broken27
Copy link
Contributor

@Broken27 Broken27 commented Jun 17, 2021

Summary

Balance "Increases plate armor coverage from 95 to 98%"

Purpose of change

A 5% chance to randomly hit plate armor joints is too high and unrealistic.

Describe the solution

It's a simple number change

Describe alternatives you've considered

None

Testing

I debugged a suit of plate, confirmed the new coverage

Additional context

None

A 5% chance to randomly hit plate armor joints is too high and unrealistic.
@anothersimulacrum anothersimulacrum added the [JSON] Changes (can be) made in JSON label Jun 17, 2021
@I-am-Erk
Copy link
Member

We understand that plate armour covers more than this, but until we have expanded armour code to allow partial hits and weakpoints in a reasonable way instead of all-or-nothing, and allowing hordes of monsters to overwhelm it, having a day-1-accessible armour suit make you functionally invulnerable to most threats in the game is not desired behaviour.

@I-am-Erk I-am-Erk closed this Jun 17, 2021
@I-am-Erk
Copy link
Member

Apparently @mlangsdorf feels differently so I'll reopen this to discuss.

@I-am-Erk I-am-Erk reopened this Jun 17, 2021
@Broken27
Copy link
Contributor Author

Broken27 commented Jun 17, 2021

We understand that plate armour covers more than this, but until we have expanded armour code to allow partial hits and weakpoints in a reasonable way instead of all-or-nothing, and allowing hordes of monsters to overwhelm it, having a day-1-accessible armour suit make you functionally invulnerable to most threats in the game is not desired behaviour.

.... how is plate armor accesible day 1? It only spawns in Mansions (That are super rare now) and Museums (That usually appear in zombie-dense town centers).

And if you want to craft one yourself, it needs Fab 8, a forge, and six different proficiencies.

(In fact, aside of roleplaying, there is currently no reason to ever make one, since by the point when you can do it you can almost certainly make survivor gear than is superior in every way).

@I-am-Erk
Copy link
Member

Mansions are a potential starting spawn location.

@Broken27
Copy link
Contributor Author

Mansions are a potential starting spawn location.

Not anymore actually.

@I-am-Erk
Copy link
Member

huh, I missed that fix. Good. Regardless, it is tangential to the main concern, which is that this is a bandaid patch that IMO isn't an appropriate fix to the issue. Since others disagree I've reopened this, but I've said my piece.

@anothersimulacrum
Copy link
Member

Perhaps the (vast majority of the) spawns of plate armor in museums and mansions could be replaced with fake/poorly maintained plate armor with inferior stats?

@I-am-Erk
Copy link
Member

Perhaps the (vast majority of the) spawns of plate armor in museums and mansions could be replaced with fake/poorly maintained plate armor with inferior stats?

That's a related issue but not the same, it should be almost unthinkable that useable battle-ready plate armour that fits you spawns. Like, in a museum you could potentially find something like "museum plate armour" that's not really wearable, but perhaps studying it can give you proficiency training in making armour, and perhaps you can upgrade it into wearable plate armour.

Spawn details aren't really a good replacement for my core issue though, which is that high coverage makes the armour close to perfect because we lack mechanisms for partial protection by armour, or for zombie hordes overwhelming you and pulling your plates off.

@ZhilkinSerg
Copy link
Contributor

Last Delivery (#39703) points player to mansion.

@Broken27 Broken27 closed this Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[JSON] Changes (can be) made in JSON
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants