Unit Option When Damaged Change Into Different Unit (Weakened Battleships)

[prastle]

@hepps as always you amaze me H because I was actually thinking the same thing. Obviously they need to be Pu related and tied together. Damage = damage. I wonder if its possible? Once again you thought of the question before I even asked it.

[Hepps]

@prastle Well they wouldn't necessarily need to be tied together or related... as long as the sample XML clearly defines the uses and limitations/potential conflicts of using both systems on the same unit... then you can have 2 independent types of damage.

[prastle]

@hepps I cant see how independent would work when only one monetary system.

[Hepps]

<option name="whenHitPointsDamagedChangesInto" value="1:germanBattleshipDamaged:true" count="1"/>
<option name="whenBombingDamagedChangesInto" value="1:germanBattleshipDamaged:true" count="1"/>

So question on this in case I am missing something here...

Even though the 2 types of damage are independent of one another.... if the unit changed into the same "Damaged" unit at the end of the current combat phase... wouldn't then either type of damage be cumulative?

ie. I send my strat. Bomber in to bomb a undamaged BB. Let's say it achieves a hit and the bombing damage option then changes the unit into a damaged ship (if there is only 1 type of damaged ship), then that new unit would by virtue of the unit change automatically be changed to a unit having only 2 remaining HP.... so then if you had also launched an attack against the same battleship... during the same pound wouldn't the unit now still have only 2 HP at the start of the naval combat? therefore basically overrighting the bombing damage?

[redrum]

@hepps It would only work that way if the damaged BB unit had say max HP of 2 instead of 3. The reason is it would only carry over the damage of the unit being replaced (if 'keepDamage'=true) which would only be bombing damage in your case. So you'd end up with a damaged BB unit with 1 bombing damage and 0 hit point damage.

[Hepps]

@redrum Ah ok. So you would then have to have 4 different types of damaged unit, causing all kinds of issues with unit changing and re-changing depending on the damage taken... that does present an issue.

[Hepps]

@hepps So using both types of damage on multi hit units is probably not a realistic goal.

[redrum]

@hepps Agree. There would need to be further thought and logic to better handle units that can take both types of damage. Probably something like a damageConvert unit option which would translate bombing damage to HP and vice-versa. So say if a BB had 3 HP and 15 max bombing damage then you could have a 1-to-5 damage conversion ratio so it is damaged evenly (ex. if it takes 5 bombing damage then would take 1 HP damage).

You'd also have to think about how repair would work as well (again some conversion ratio so both are repaired at the same time or separate repairs). It gets pretty complicated pretty quickly.

[Hepps]

@redrum Agreed.

Probably best if those two conversions were independent of one another.

[redrum]

@hepps Initial PR to address the damage (not repair) portion of this feature: https://github.com/triplea-game/triplea/pull/2689

Tested it by editing TWW to add unit, unit image, and below unit attachments:

<attachment name="unitAttachment" attachTo="germanBattleship" javaClass="games.strategy.triplea.attachments.UnitAttachment" type="unitType">
  <option name="movement" value="2"/>
  <option name="attack" value="7"/>
  <option name="defense" value="8"/>
  <option name="canBombard" value="true"/>
  <option name="isSea" value="true"/>
  <option name="hitPoints" value="2"/>
  <option name="bombard" value="6"/>
  <option name="requiresUnits" value="germanDocks:germanFactory"/>
  <option name="consumesUnits" value="1:germanHull"/>
  <option name="whenHitPointsDamagedChangesInto" value="1:true:germanBattleshipDamaged"/>
</attachment>
<attachment name="unitAttachment" attachTo="germanBattleshipDamaged" javaClass="games.strategy.triplea.attachments.UnitAttachment" type="unitType">
  <option name="movement" value="1"/>
  <option name="attack" value="3"/>
  <option name="defense" value="4"/>
  <option name="canBombard" value="true"/>
  <option name="isSea" value="true"/>
  <option name="hitPoints" value="2"/>
  <option name="bombard" value="3"/>
</attachment>

[Hepps]

@redrum AWESOME!!!!!

[redrum]

Initial PR merged and can be tested on https://github.com/triplea-game/triplea/releases/tag/1.9.0.0.7804

[redrum]

Second PR to address repairing them: https://github.com/triplea-game/triplea/pull/2691

Tested with this XML:

<attachment name="unitAttachment" attachTo="germanBattleship" javaClass="games.strategy.triplea.attachments.UnitAttachment" type="unitType">
  <option name="movement" value="2"/>
  <option name="attack" value="7"/>
  <option name="defense" value="8"/>
  <option name="canBombard" value="true"/>
  <option name="isSea" value="true"/>
  <option name="hitPoints" value="2"/>
  <option name="bombard" value="6"/>
  <option name="requiresUnits" value="germanDocks:germanFactory"/>
  <option name="consumesUnits" value="1:germanHull"/>
  <option name="whenHitPointsDamagedChangesInto" value="1:true:germanBattleshipDamaged"/>
</attachment>
<attachment name="unitAttachment" attachTo="germanBattleshipDamaged" javaClass="games.strategy.triplea.attachments.UnitAttachment" type="unitType">
  <option name="movement" value="1"/>
  <option name="attack" value="3"/>
  <option name="defense" value="4"/>
  <option name="canBombard" value="true"/>
  <option name="isSea" value="true"/>
  <option name="hitPoints" value="2"/>
  <option name="bombard" value="3"/>
  <option name="whenHitPointsRepairedChangesInto" value="0:true:germanBattleship"/>
</attachment>

[Hepps]

@redrum So pumped!

[redrum]

@hepps Nice. Those are looking good.

[redrum]

Second PR merged so all changes can be tested in this or later releases: https://github.com/triplea-game/triplea/releases/tag/1.9.0.0.7850

[General_Zod]

@redrum @hepps First of all very cool feature. Does the battle calculator give an accurate outcome when these units are involved?

[redrum]

@general_zod That is an excellent question, that I actually don't know the answer to. I believe that it should since it actually does simulate battles. It would be interesting to make a very small test to prove or disprove it though.

[alkexr]

Now that I think of it...

<attachment name="unitAttachment" attachTo="hydraHead" javaClass="games.strategy.triplea.attachments.UnitAttachment" type="unitType">
  <option name="movement" value="0"/>
  <option name="attack" value="3"/>
  <option name="defense" value="3"/>
  <option name="whenHitPointsDamagedChangesInto" value="2:false:hydraHead"/>
</attachment>

[Hepps]

@alkexr