Experts: AutoRisk

[Automated-Trading]

AutoRisk:

Shows your position size through a comment from any symbol chart based on a risk percent on daily ATR, you can use either balance or equity.

Author: David Diez

[[Deleted]]

Automated-Trading:

AutoRisk:

Author: David Diez

I think this should be called 'LotSizer', don't you agree?

[Alain Verleyen]

Poor quality code. Call iATR on each tick, no error checking

[David Diez]

Alain Verleyen #:
Poor quality code. Call iATR on each tick, no error checking

But the risk-management strategy is still good.

[Ryan L Johnson]

Alain Verleyen #:
Poor quality code. Call iATR on each tick, no error checking

Does the iATR call inside OnTimer() actually return data on each tick? 🧐

Note that OnTimer() is set to run on each second in the code.

[Alain Verleyen]

Ryan L Johnson #:

Does the iATR call inside OnTimer() actually return data on each tick? 🧐

Note that OnTimer() is set to run on each second in the code.

The code was changed since I commented.

[Ryan L Johnson]

Alain Verleyen #:
The code was changed since I commented.

In that case, well done on contributing to help development. I see that another contribution led to changing the name of the indicator.

Personally, I believe that adding a horizontal stoploss line would make the indicator more useful.