-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Expand implementation of liquid ammo #72271
base: master
Are you sure you want to change the base?
Conversation
Am I correct in understanding that this attack is similar to a rifle attack? So in one shot, an attack can only hit one body part? |
No, the point is to re-use the shotgun spread implementation |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not bump or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered. |
I noticed that a "bad scent attack" already exists in the form of the {
"id": "skunk_squirt",
"type": "SPELL",
"name": { "str": "Skunk Spray" },
"description": "Skunk spray defense mechanism.",
"valid_targets": [ "hostile" ],
"effect": "attack",
"shape": "cone",
"min_range": 4,
"max_range": 4,
"//": "Duration is 3-5 hours, can't seem to write it in hours, had to write it in moves which is terrible",
"min_duration": 1080000,
"max_duration": 1800000,
"flags": [ "RANDOM_DURATION" ],
"effect_str": "skunk_spray"
} Which applies its effects mostly using enchantments: {
"type": "effect_type",
"id": "skunk_spray",
"name": [ "Skunk Spray" ],
"desc": [ "You were sprayed by a skunk!" ],
"apply_message": "You're covered in a foul smelling fluid!",
"rating": "bad",
"max_duration": "5 hours",
"base_mods": { "per_mod": [ -1 ], "int_mod": [ -1 ], "vomit_chance": [ 2000 ] },
"enchantments": [
{
"values": [
{ "value": "SOCIAL_PERSUADE", "add": -10 },
{ "value": "SOCIAL_LIE", "add": -4 },
{ "value": "SOCIAL_INTIMIDATE", "add": 2 },
{ "value": "UGLINESS", "add": 5 },
{ "value": "SCENT_MASK", "add": 300 }
]
}
]
} I'm curious if any of this will be changed or if it will be considered a separate thing. |
c9daa91
to
d1b0452
Compare
Everything looks reasonable so far, have you tried adding shot parameters and seeing if it does multi-hit as expected? |
This isn't urgent, but at some point we'll probably want a new set of automated test cases. |
c35a4da
to
dd5e654
Compare
dd5e654
to
ebf4de4
Compare
Summary
Features "Expand implementation of liquid ammo"
Purpose of change
Reimplement liquid attacks from #71964 using liquid ammo instead of spells
Describe the solution
Character
- [x] Add option to only apply effect if a specific body part is touched- [ ] Instead of above option add a filter by bodypart type likesensor
for exemple- [ ] Make a dedicated liquid soundNo need, soundpack can already set sounds per gun idsIRRITANT_IMMUNE
to more armors and exoskeletons #77524 since armor has no reason to block skin irritation other than actually stoping the liquid going throughDescribe alternatives you've considered
Testing
2024-03-09.19-37-26.mp4
Additional context
Need #72169 merged first
This will probably make boomer attacks undodgeable, setting projectile speed to 1à instead of 1000 for liquid ammo doesn't seem to help from a quick test