Target5150 / MoYu_Server_Stupid_Plugins

L4D2 Target Cat Project
GNU General Public License v3.0
80 stars 21 forks source link

l4d2_fix_deathspit and l4d2_spit_spread_patch #15

Closed nikita1824 closed 2 years ago

nikita1824 commented 2 years ago

Hello, as I understand, we have to delete l4d2_fix_deathspit plugin if we use l4d2_spit_spread_patch. But same fix in your plugin doesn't work properly.

jensewe commented 2 years ago

l4d2_fix_deathspit functionality is included so yes. I didn't dig into how proper disabling puddle should be, and just copy-pasted codes from l4d2_fix_deathspit. So is it still doing damage?

jensewe commented 2 years ago

I think i got it, and seems it's solved with some changes. Will push an update soon.

nikita1824 commented 2 years ago

Hello, thank you for your attention to this problem but invisible deathspit still occurs even after your recent updates.

jensewe commented 2 years ago

Which version are you using? It will be appreciated if you can be more specific on how to reproduce it.

nikita1824 commented 2 years ago

I'm sorry for not providing this information. Plugin version 1.8 Sourcemod version 1.10.0.6495 ConVar settings:

To reproduce this problem I just spawned a spitter bot and killed it with any gun. Walking around deathspit not touching visible textures caused me a damage anyway.

jensewe commented 2 years ago

Hmm I don't think it's the one called invisible spit. You might want to check if the death spit is on uneven surface (small angled ramp for example), where part of the spit cannot be seen.

Perhaps it comes in inconsistently but it seems I managed to be damage-free as near as this for several times:

image

nikita1824 commented 2 years ago

Here is demo. I hope it will be usefull.

jensewe commented 2 years ago

Thanks for the video. Though I begin to wonder how vanilla death spit performs like, as I went through l4d2_fix_deathspit again which seems likely to prevent partial deathspit from being activated consistently. So probably I've been wrong about what the invisible spit means. It would be great that you can do me a favor to confirm whether vanilla death spit has the same issue.

Edit: wrong touch LUL

nikita1824 commented 2 years ago

Yeah, vanilla deathspit has the same (or may be even worse) issue. Here is demo which is recorded on local server with sv_cheats 1.

jensewe commented 2 years ago

Thanks a lot for keeping in contact, I've committed an update that should actually include the fix from the other plugin. Please see if that resolves the issue.

nikita1824 commented 2 years ago

Hello, I want to thank you for resolving this problem, now it works fine.

nikita1824 commented 2 years ago

Hello again, too bad that I had to reopen issue about this plugin, now we have another problem. It's just error log spam but it's unpleasant to get them.

L 03/12/2022 - 18:30:36: SourceMod error session started
L 03/12/2022 - 18:30:36: Info (map "l4d2_stadium1_apartment") (file "/home/srcds/.steam/steamcmd/l4d2server/left4dead2/addons/sourcemod/logs/errors_20220312.log")
L 03/12/2022 - 18:30:36: [SM] Exception reported: Invalid memory access
L 03/12/2022 - 18:30:36: [SM] Blaming: l4d2_spit_spread_patch.smx
L 03/12/2022 - 18:30:36: [SM] Call stack trace:
L 03/12/2022 - 18:30:36: [SM]   [1] Line 208, D:\scripting\_plugins\l4d2_spit_spread_patch\scripting\l4d2_spit_spread_patch.sp::OnEntityCreated
L 03/12/2022 - 18:30:40: [SM] Exception reported: Invalid memory access
L 03/12/2022 - 18:30:40: [SM] Blaming: l4d2_spit_spread_patch.smx
L 03/12/2022 - 18:30:40: [SM] Call stack trace:
L 03/12/2022 - 18:30:40: [SM]   [1] Line 208, D:\scripting\_plugins\l4d2_spit_spread_patch\scripting\l4d2_spit_spread_patch.sp::OnEntityCreated
...
L 03/12/2022 - 18:50:28: Error log file session closed.

I'm not quite sure about this type of logs but it seems like if you try to get 7th symbol of classname string and its length is shorter, we get these errors.

jensewe commented 2 years ago

Good find! Thanks a lot, will commit an update soon.