Skip to content
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

Merged
merged 4 commits into from
Jul 25, 2023

Conversation

ASpoonPlaysGames
Copy link
Contributor

@ASpoonPlaysGames ASpoonPlaysGames commented Jul 22, 2023

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 and npc_damage_far_distance vars for NPCs

closes #672

@Zanieon
Copy link
Contributor

Zanieon commented Jul 23, 2023

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.

@Alystrasz
Copy link
Contributor

Could you describe how to ensure your PR fixes the issue?
(here, how to spawn a NPC and give him the correct weapon)

@ASpoonPlaysGames
Copy link
Contributor Author

Could you describe how to ensure your PR fixes the issue? (here, how to spawn a NPC and give him the correct weapon)

Easiest way, drop a titan, get in, spawn arc cannon with give and get out of the titan, then let it shoot some grunts or something

Copy link
Contributor

@Alystrasz Alystrasz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This pull request effectively restores weapon's expected behavior while on the hands of NPCs.

Before change

My titan fires at soldiers, without dealing any damage to them.

image

image

After change

My titan makes soldiers explode with the weapon.

image

image

Copy link
Member

@F1F7Y F1F7Y left a 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

@F1F7Y F1F7Y merged commit 18298d2 into R2Northstar:main Jul 25, 2023
3 checks passed
@ASpoonPlaysGames
Copy link
Contributor Author

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

GeckoEidechse pushed a commit that referenced this pull request Jul 25, 2023
Fixes NPCs using mp_titanweapon_arc_cannon not dealing damage to other entities

(cherry picked from commit 18298d2)
@Alystrasz
Copy link
Contributor

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

Can you create an issue for this? Are you willing to work on it?

@ASpoonPlaysGames
Copy link
Contributor Author

Can you create an issue for this?

Sure

Are you willing to work on it?

Maybe

@Dinorush
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

NPCs using the Arc Cannon don't charge it up
6 participants