Skip to content

Fix cotton resistances#86518

Open
Maleclypse wants to merge 1 commit intoCleverRaven:masterfrom
Maleclypse:Fix-#86499
Open

Fix cotton resistances#86518
Maleclypse wants to merge 1 commit intoCleverRaven:masterfrom
Maleclypse:Fix-#86499

Conversation

@Maleclypse
Copy link
Copy Markdown
Member

Summary

None

Purpose of change

Fixes #86499

Describe the solution

Adjusts cotton material resistances

Describe alternatives you've considered

none

Testing

Additional context

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON <Bugfix> This is a fix for a bug (or closes open issue) astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Apr 16, 2026
@GuardianDll
Copy link
Copy Markdown
Member

GuardianDll commented Apr 16, 2026

-------------------------------------------------------------------------------
armor_protection
  no protection from physical, no protection from environmental
-------------------------------------------------------------------------------
../tests/iteminfo_test.cpp:1412
...............................................................................

../tests/iteminfo_test.cpp:1287: FAILED:
  REQUIRE( armor.resist( damage_bullet ) == Approx( bullet ) )
with expansion:
  0.05f == Approx( 0.200000003 )
with message:
  armor.typeId().str() := "test_longshirt"

-------------------------------------------------------------------------------
armor_protection
  complex protection from physical and environmental damage
-------------------------------------------------------------------------------
../tests/iteminfo_test.cpp:1527
...............................................................................

../tests/iteminfo_test.cpp:1287: FAILED:
  REQUIRE( armor.resist( damage_bullet ) == Approx( bullet ) )
with expansion:
  10.16667f == Approx( 10.6666698456 )
with message:
  armor.typeId().str() := "test_complex_tanktop"

-------------------------------------------------------------------------------
armor_stats
-------------------------------------------------------------------------------
../tests/iteminfo_test.cpp:1293
...............................................................................

../tests/iteminfo_test.cpp:1287: FAILED:
  REQUIRE( armor.resist( damage_bullet ) == Approx( bullet ) )
with expansion:
  0.025f == Approx( 0.1000000015 )
with message:
  armor.typeId().str() := "tshirt"

@BruhddyTheThird
Copy link
Copy Markdown
Contributor

Does this actually fix #86499? I.e. what are the resistance values of the "Ichcahuipilli" after this change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ichcahuipilli is likely overpowered

3 participants