-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
[RDY] Adds weapon chemical launcher and new fungicidal gas type as ammo #12765
Changes from all commits
3780b1a
415cbf0
faebdbc
61a0b72
28c5b0c
401c156
aec4e01
09d4f44
39770ce
0dfd529
efbf03a
c77226d
cfd851c
7324aef
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2974,7 +2974,7 @@ | |
"environmental_protection" : 20, | ||
"encumbrance" : 37, | ||
"bashing" : -5, | ||
"flags" : ["VARSIZE", "WATERPROOF", "RAINPROOF", "RAD_PROOF","ELECTRIC_IMMUNE", "OUTER"], | ||
"flags" : ["VARSIZE", "WATERPROOF", "RAINPROOF", "RAD_PROOF", "ELECTRIC_IMMUNE", "GAS_PROOF", "OUTER"], | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. GAS_PROOF would be better as an environmental protection formula (so that you could partially resist it), but it's OK as a tag for now. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I do plan on adding a variety of ammo for the thrower. Will keep that in mind. |
||
"coverage" : 100, | ||
"material_thickness" : 2 | ||
}, | ||
|
@@ -2999,7 +2999,7 @@ | |
"environmental_protection" : 20, | ||
"encumbrance" : 45, | ||
"bashing" : -5, | ||
"flags" : ["VARSIZE", "WATERPROOF", "RAINPROOF", "STURDY", "RAD_PROOF", "ELECTRIC_IMMUNE", "OUTER"], | ||
"flags" : ["VARSIZE", "WATERPROOF", "RAINPROOF", "STURDY", "RAD_PROOF", "ELECTRIC_IMMUNE", "GAS_PROOF", "OUTER"], | ||
"coverage" : 100, | ||
"material_thickness" : 3 | ||
}, | ||
|
@@ -6601,7 +6601,7 @@ | |
"environmental_protection" : 20, | ||
"encumbrance" : 50, | ||
"bashing" : -5, | ||
"flags" : ["VARSIZE", "WATERPROOF", "RAINPROOF", "STURDY", "SUN_GLASSES"], | ||
"flags" : ["VARSIZE", "WATERPROOF", "RAINPROOF", "GAS_PROOF", "STURDY", "SUN_GLASSES"], | ||
"coverage" : 100, | ||
"material_thickness" : 5 | ||
}, | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was having trouble with assigning a more general chemical_spray ammo-type. I wanted something like this:
{
"type" : "ammunition_type",
"id" : "chemical_spray",
"name" : "sprayable chemical",
"default" : "gas_fungicidal"
}
But it throws a debug-message, saying gas_fungicidal is not a valid ammo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like you only changed it in ammo_types, but not in ammo.json: "gas_fungicidal" had the old "ammo_type" field.
"ammo" field refers to the ammo type id. So if you're changing ammo id to "chemical_spray", the fungicidal gas needs
"ammo_type": "chemical_spray"
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I did that ;) Still did not work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh forgot: the same in the thrower.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's it! Also, it makes much more sense, now that I think about it.