I would agree Jan that is what is intended, but surely the multiplier should be the instrument price (as in DJI 21,600 currently)?  Reason: positionperf is the % gain of postion(n).

As an aside … this is interesting for me to see how you guys (far far better than me at coding) do not find it easy to interpret another author’s code?