TheCookieLab

Sure, your willingness to take on the effort is appreciated, and if you can show comparable performance without the cache then we may not need it after all.

@francescopeloi you are correct, the indicator should be aware of all the values. There actually used to be a ZigZagIndicator class in the project but it was removed at some...

> thanks for your answer @TheCookieLab . I found that indicator you posted, but couldn't make it work properly, so started from scratch. I think this indicator can be implemented...

Agreed, having this as consistent behavior would be ideal. How do you see us implementing this - just returning `NaN` directly or should we use the `UnstableIndicator` to do so?...

@hhashim1 are you able/willing to take this on and create a PR?

> Adding this to the system would produce a lot of checking for `isUnstable` and/or `isNaN` because ta4j allows to arbitrary combine indicators with other indicators. For example the `EMAIndicator`...

Try incrementally reducing the barCount from 10 and tracing through to get a better understanding of the mechanics. ConvergenceDivergenceIndicator uses the IsRising and IsFalling rules which may not perform how...

Good catch, do you want to create a PR to resolve this issue? Thanks. On Wed, Feb 14, 2024 at 3:38 PM Cashmeiser ***@***.***> wrote: > *I have checked existing...

Typically contributing to open source projects is done via pull requests. Please see the contribution guide here: https://ta4j.github.io/ta4j-wiki/How-to-contribute As for the implementation, to be consistent with the rest of the...

There is not, at least not out of the box. TradingRecord only allows one open position at a time so you’d have to hack together multiple trading records to manage...