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

Correct targeting for small arms dmg type #149

Open
pvutov opened this issue Jun 14, 2020 · 0 comments
Open

Correct targeting for small arms dmg type #149

pvutov opened this issue Jun 14, 2020 · 0 comments
Labels
bug Something isn't working

Comments

@pvutov
Copy link
Collaborator

pvutov commented Jun 14, 2020

Turret system currently wont lock onto soft vehicle targets if a unit only has small arms weapons. The issue is in Ammo.cs, GetRangeAgainstTargetType. However it also has to do with how we currently pick targets. Ultimately the issue is that we should only pick vehicle targets if they are facing us with a <2AV side, and drop them if they stop facing in this way (currently we never really drop the current target).

Obsolete, old description:
When defining the damage types I forgot to include small arms.

We need to add a small arms option to the unit config, a small arms damage type in Ammo.cs, and a small arms formula in ArmorComponent.cs. This damage type doesnt have an explosion radius, does 10% damage against 1AV and no damage to 2AV.

Might need a heavy small arms type which follows the same rules but does 50% damage against 1AV.

Also we might need to do this with an AoE tag if we want to allow cluster..

@pvutov pvutov added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Jun 14, 2020
@pvutov pvutov changed the title Implement small arms dmg type Correct targeting for small arms dmg type Jun 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant