Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test_imi failed #14

Open
wony-zheng opened this issue Dec 26, 2023 · 1 comment
Open

test_imi failed #14

wony-zheng opened this issue Dec 26, 2023 · 1 comment
Assignees

Comments

@wony-zheng
Copy link

ta_regtest V0.6.0-dev (Dec 26 2023 08:25:05) - Regression Tests of TA-Lib code

Testing utility functions
Testing Abstract interface
Testing the TA functions
MATH,VECTOR,DCPERIOD/PHASE,TRENDLINE/MODE: Testing...done.
All Moving Averages: Testing...done.
AROON,CORREL,BETA: Testing...done.
CCI,WILLR,ULTOSC,NATR: Testing...done.
BOP,AVGPRICE: Testing...done.
RSI,CMO: Testing...done.
IMI: Testing...
Fail: Value out of tolerance range (0.429738,0.00458716)
Fail: doRangeTestFixSize diff data for idx=0 (4.418833e+01,7.748777e+01)
Fail: doRangeTestFixSize (232,232,232,1,1)
Fail: doRangeTestFixSize refOutBeg,refOutNbElement (4,248)
Fail: Diff 75.358 %
Fail: Using unstable period 208
Failed: For output #1 of 1
D:\WorkCopy\ta-lib\src\tools\ta_regtest\ta_test_func\test_imi.c Failed Test #0 (Code=161)

@mario4tier mario4tier self-assigned this Nov 8, 2024
@mario4tier
Copy link
Member

Summary

Thanks for reporting this. I did commit a work-around and the test should pass now.

Details

This was a false negative. The test itself is incorrectly using a too small tolerance range.

For now, I did disable the invalid test, but will revisit this later to test with a wider tolerance appropriate to the algo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants