ta-rs icon indicating copy to clipboard operation
ta-rs copied to clipboard

Published 20 hours ago • verified publisher icon greyblake

Add DX/DMI & ADX

Open virtualritz opened this issue 2 years ago • 2 comments

Requires DM-/DM+, DI-/DI+ first.

I started implementing all of them in my fork as I need them for the project I'm working on. While they're all there now I haven't yet verified them and/or wrote tests.

I will update this issue and open a PR once that is done.

In the meantime I'm happy for feedback – just opening this issue so others can benefit/don't do work twice.

virtualritz avatar Jul 20 '22 17:07 virtualritz

Is this complete? if so, would you mind going ahead and raising a PR? - i am trying to get a batch of PRs ready and see if we can get the TA crate updated

SeanEClarke avatar Oct 02 '23 17:10 SeanEClarke

I'm not comfortable submitting a PR until #70 is resolved.

In general, I think it would give users confidence if the original TA-lib test data was used to verify all indicators in ta-rs work as in the original.

virtualritz avatar Oct 09 '23 10:10 virtualritz