-
Notifications
You must be signed in to change notification settings - Fork 118
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
Fix Arc Cannon not dealing damage when used by NPCs #666
Conversation
Tested, it works, though damage for near and far value should probably be the same for NPCs, they don't charge it so they will always pick lowest damage value, currently it kills grunts and pilots in one shot, but Stalkers or higher it doesnt, it also does pity damage against other Titans. EDIT: I tried adjusting Titan armor damage to the same of Charge Rifle when used by NPCs, it seems a fair value to use. |
Could you describe how to ensure your PR fixes the issue? |
Easiest way, drop a titan, get in, spawn arc cannon with |
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.
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.
Changes look good, works in testing
It should be noted that I have been told that NPC titans do not charge up the arc cannon, meaning that they tend to do rather shite against other titans. It has been suggested that we should up the minimum heavy armour damage for npcs, which would alleviate the issue somewhat |
Fixes NPCs using mp_titanweapon_arc_cannon not dealing damage to other entities (cherry picked from commit 18298d2)
Can you create an issue for this? Are you willing to work on it? |
Sure
Maybe |
Could alternatively do what Railgun does and make NPCs get a flat amount of charge when shooting. Only matters vs adjusting minimum damage if charge affects FX though. |
This might work, idk.
Literally just made this in github's text editor, someone will need to test.
Damage for NPCs is set to be the same as by players, not exactly sure if other weapons follow this, but it's not like the auto-titans need a nerf.
Also makes the weapon script use the
npc_damage_near_distance
andnpc_damage_far_distance
vars for NPCscloses #672