Post by ChezJfrey in Discrepancy in sail damage calculations

Viewing post in Discrepancy in sail damage calculations

Upon further review, you are correct that the engine does update .SP attribute. It calculates .SP as an integer percentage of holes in each sail, relative to the maximum holes allowed for each sail type, as follows:

Triangle maxHole = 10
Other maxHole = 12
Default maxSP = 100
numberholesinsail can never exceed maxHole for sail type

fSP = maxSP * (maxHole - numberholesinsail) / maxHole;

.SP = fSP as integer

The engine does not record the .dmg decimals in the scripts and has no knowledge of those values

Like Reply

Qwerry, St.3 years ago

OK
Now I realize that you completly didn't understand what I need.
:(

Like Reply

ChezJfrey3 years ago

Well I'm not going to change how the engine calculates and updates Ship.Sail.SP value.

The scripts update that number using .dmg, in function CalculateShipSP. The engine uses a percentage based on numHoles/maxHoles; both of those are available in the .hc and .mhc attributes of the sails. You can either change CalculateShipSP to use the same calculation I provided, instead of .dmg, so they will always match. Or, you can store CalculateShipSP in a different attribute within your scripts (example Ship.Sail.SP2) and use that new attribute everywhere in your scripts when you repair, but then adjust the engines hole count to match your desired SP value that you calculate in the scripts, by using DeleteOneSailHole, to inform the engine to adjust the hole count that will cause the engine to calculate a similar Ship.Sail.SP value when it employs holes/maxHoles.

Like Reply

Qwerry, St.3 years ago

discrepancies occur when the mast falls.

all information about how many sails are considered attributed to this fallen mast is not available in the script part

but nevertheless, when mast falls, chr.ship.sp data is instantly updated in BattleInterface and this is not done by the calculateshipsp function.

I need information about what percentage of all ship sails the engine subtracts when the mast falls. and how I can get this persent in scripts.

just forget about sail holes.

Like Reply

ChezJfrey3 years ago (1 edit)

You can't ignore sail holes because that is precisely the measurement the engine is using to set the .SP value and you want to know how to arrive at that figure. The engine will send each sail through event "DoSailHole" (function ProcessSailDamage()). During a mast fall, sMastName will not equal "*" and holeCount will equal maxHoleCount. You could possibly record the number of holes for each fallen mast.

An idea that might work in ProcessSailDamage:

if(sMastName!="#")   {
arSail.mastFall = sMastName;
if(CheckAttribute(chref, "mastFalls." + sMastName))
    chref.mastFalls.(sMastName) = sti(chref.mastFalls.(sMastName)) + holeCount;
else
    chref.mastFalls.(sMastName) = holeCount;
}

With that information, plus the example to arrive at the same engine .SP value below, should hopefully help.

Also, an example to arrive at the same .SP the engine calculates:

float CalculateShipSP(ref chref)
{
    float fSP = GetCharacterShipSP(chref);
    aref arRoot,arGroup,arSail;
    int q,n,i,j;
    int hc = 0;

   makearef(arRoot, chref.ship.sails);
    q = GetAttributesNum(arRoot);
    for(i=0; i<q; i++)
    {
        arGroup = GetAttributeN(arRoot,i);
        n = GetAttributesNum(arGroup);
        for(j=0; j<n; j++)
        {
            arSail = GetAttributeN(arGroup,j);
            if(!CheckAttribute(arSail, "dmg"))
            {
                arSail.dmg = 0.0;
            }
            if(CheckAttribute(arSail, "hc")) //Added for test to accumulate holeCount for each sail
            {
                hc += sti(arSail.hc);
            }
            fSP -= stf(arSail.dmg);
        }
    }

   if(fSP<0.0) fSP = 0.0;
    int st = GetCharacterShipType(GetCharacter(sti(chref.index)));
    ref shref = GetRealShip(st); //This is .maxSP sent to engine in BI_objRetValue.maxSP from procGetSailTextureData() event
    int ssP = sti(chref.ship.sails); //Engine stores cumulative maxHoles in .ship.sails root attribute
    trace("ship.sails " + chref.ship.sails); //Engine stores cumulative maxHoles in .ship.sails root attribute
    trace("hc " + hc); //Accumulated holeCount
    trace("RSP = " + shref.sp); //This is the maxSP the engine will use to figure .SP
    trace("CSP = " + chref.ship.sp); //This is engine recorded .SP
//ssP - hc is maxHoles - holecount
    trace("SPtest = " + sti(shref.sp) * (ssP - hc) / ssP);
    trace("fSP = " + fSP);
    return fSP;
}

I just tested the following, by calling: CalculateShipSP(pchar);

This was the output in compile.log:

ship.sails 126
hc 9
RSP = 350
CSP = 325
SPtest = 325
fSP = 311.4

CSP is the engine recorded ship.SP
SPTest is figured from realship.SP * (chref.ship.sails - cumulative .hc) / chref.ship.sails

Like Reply

Qwerry, St.3 years ago

as expected, it didn't work when the mast fell.
=
if (sMastName!="#")
{
arSail.mastFall = sMastName;
if(CheckAttribute(chref, "mastFalls." + sMastName))
chref.mastFalls.(sMastName) = sti(chref.mastFalls.(sMastName)) + holeCount;
else chref.mastFalls.(sMastName) = holeCount;
}
code doesn't work at all. Probably the name of the mast is never different from *.
I checked breakage of mast with another ship collision and from shelling. And this section of code has never been executed.

So I don't have any new information for recalculations.

Like Reply

ChezJfrey3 years ago

You appear frustrated, but this exchange of information has proved useful. What I described in my previous post is what I saw when I mentally traced the engine source code logic and that event should be called when a mast falls. However, given your claim that event does not fire properly, I physically traced the logic during actual execution and found you are correct, that the event does not execute during mast fall. But it should execute, and I found where the original developers intended it to perform, but they made a mistake, which I was able to correct.

I just updated Maelstrom, and now during a mast fall, the information for the fallen mast, and its relevant sails, will now be sent to this event.

Like Reply

itch.io

Viewing post in Discrepancy in sail damage calculations