// Definition of code parameters
DEFPARAM CumulateOrders = true // Cumulating positions deactivated

// Conditions to enter long positions
c1 = (close > high[12])
indicator1 = Average[50](close)
c2 = (close > indicator1)

// Buying the market conditions
IF c1 AND c2 THEN
BUY 1 SHARES AT MARKET
ENDIF

// Cummulate orders until max position achieved set limits
IF LONGONMARKET AND COUNTOFLONGSHARES < 3 AND Open > Close[1] THEN
BUY 1 SHARES AT MARKET
ENDIF

// Conditions to exit long positions
c3 = (close < low[14])
c4 = (close < indicator1)
IF c3 OR c4 THEN
SELL AT MARKET
ENDIF

// Stops and targets
SET STOP %LOSS 4

Hi Josh, welcome…

Here’s the code from your screenshot…

The issue with your system not respecting the maximum position size lies in line 10 of your code…

When the conditions c1 and c2 are true, a position is opened. However, these conditions can remain true for multiple bars (days), which causes the system to keep opening new positions as long as the conditions hold…

The solution is to replace line 10 with the following condition:

If NOT LongOnMarket and c1 and c2 then

This ensures that a new position is only opened when no position is already active…