search

f-cksociety / backtesting-trading-engine-pinecoders

Masterpiece of PineCoders script https://www.tradingview.com/script/dYqL95JB-Backtesting-Trading-Engine-PineCoders/ in v3, this is the project to upgrade it to v4-v5 and implement dynamic variable alerts message from within an indicator() alertcondition as discussed here https://t.me/pinescripters/31982
Mozilla Public License 2.0
9 stars 3 forks source link

v5 needs to touch for ta.change. #4

Closed HeyMrRobot closed 2 years ago

HeyMrRobot commented 2 years ago

As the notice/error suggests,

line 1018: The function 'ta.change' should be called on each calculation for consistency. It is recommended to extract the call from this scope. line 1464: The function 'ta.change' should be called on each calculation for consistency. It is recommended to extract the call from this scope. line 1526: The function 'ta.change' should be called on each calculation for consistency. It is recommended to extract the call from this scope. line 1527: The function 'ta.change' should be called on each calculation for consistency. It is recommended to extract the call from this scope. line 1528: The function 'ta.change' should be called on each calculation for consistency. It is recommended to extract the call from this scope. line 1846: 'alertcondition()' has no effect inside strategies. Script 'BTE v5' has been saved

HeyMrRobot commented 2 years ago

A proper issue with linked lines added https://github.com/f-cksociety/backtesting-trading-engine-pinecoders/issues/5