Support for AA Attacks

redrum

@frostion @alkexr I think both are good ideas but also fairly involved so hard to say if/when.

Multiple AA attacks per unit type would mean having separate aaAttachments similar to supportAttachments. Then either having to convert existing AA unit options to generate an aaAttachment in the engine (like isArtillery generates a supportAttachment) or having to mass convert maps so that the existing AA unit options could be removed. Both of these options are pretty painful.
AA support is probably a bit simpler and would just need some options in the support attachment to indicate its for AA instead of normal attack. Then logic added to check and apply support attachments to AA attacks. I don't think this change would impact compatibility since they would just be new options.

I don't think it matters that much the order they are implemented in as you could come back to add a supportAttachment option later on to indicate which AA attack per unit type if you did #2 first. I definitely lean towards tackling #2 first as its significantly easier (probably days vs weeks).

These also probably initially would not be taken into account by the default battle calc casualty selection though it already doesn't handle AA attacks and limited handling of support attachments.

redrum

So coming back to this.

Should support for AA attacks only be able to affect the strength or also the number of rolls?

@Frostion's example looks to re-use the dice parameter to specify it influences AA strength rather than regular strength or rolls.

<option name="dice" value="AA"/>

Should we consider allowing it to change the rolls as well? So something like:

<option name="dice" value="AAstrength"/>
<option name="dice" value="AAroll"/>

Frostion

Would the answer to the question not be a matter of "the more options the merrier"? I think that I initially just thought about strength to the dice, but adding dice could of course also be wanted in some situations.

If a unit was to have more than one AA attachment, then of course it would also be necessary to specify what AA type should be boosted/nerfed.

redrum

@frostion Probably. Just wondering if map makers would want to change number of rolls for AA attacks but generally its probably a good option to have.

I'm planning to bundle the type of AA with the addition of multiple AA attack per unit with the other feature request. Figure no need to put it in prematurely.

redrum

Here is the PR: https://github.com/triplea-game/triplea/pull/4350

AT Support

    <attachment name="supportAttachmentATSupportgerman" attachTo="germanATSupport" javaClass="games.strategy.triplea.attachments.UnitSupportAttachment" type="unitType">
      <option name="unitType" value="germanAntiTankGun"/>
      <option name="faction" value="allied"/>
      <option name="side" value="defence"/>
      <option name="dice" value="AAstrength"/>
      <option name="bonus" value="2"/>
      <option name="number" value="3"/>
      <option name="bonusType" value="ATBonus"/>
      <option name="impArtTech" value="false"/>
      <option name="players" value="Germany"/>
    </attachment>

AT Counter Support

    <attachment name="supportAttachmentATCounterAmerican" attachTo="americanATCounter" javaClass="games.strategy.triplea.attachments.UnitSupportAttachment" type="unitType">
      <option name="unitType" value="germanAntiTankGun"/>
      <option name="faction" value="enemy"/>
      <option name="side" value="offence"/>
      <option name="dice" value="AAstrength"/>
      <option name="bonus" value="-10"/>
      <option name="number" value="3"/>
      <option name="bonusType" value="ATBonus"/>
      <option name="impArtTech" value="false"/>
      <option name="players" value="Usa"/>
    </attachment>

Cernel

If the other thing of allowing multiple AA attacks per unit can be done fast (I can help with the conversion of existing maps), wouldn't be better to keep this frozen until that is done first, so this can be tailored on that done?

redrum

@Cernel Shouldn't matter much since only a few advanced maps will consider using this so would be easy to update them if necessary. I also have no idea if or when I'll add multiple AA attacks per unit.

Cernel

@redrum Ok. Seemed to me that multiple was way more important than support, but I guess also the complexity is bigger.

Frostion

My new Warcraft map will make use of the AA support feature, and also the multiple AA if it becomes available But testing is a bit difficult right now as I am still building the map.

redrum

This is now merged into the latest pre-release and can be tested: https://github.com/triplea-game/triplea/releases/tag/1.10.0.0.13130

redrum

Added info and example to POS2: https://github.com/triplea-maps/the_pact_of_steel/pull/23/files

beelee

@redrum thank you