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

update problem #105

Open
redquantum opened this issue Feb 17, 2020 · 3 comments
Open

update problem #105

redquantum opened this issue Feb 17, 2020 · 3 comments

Comments

@redquantum
Copy link

Hello dear friend,

CCI cannot calculate correctly after the last update.

Thank you

@karlwancl
Copy link
Owner

@redquantum The CCI calculation may have different definition according to different web site, I get a similar issue #103 on the CCI calculation and has fixed it once and pass the test case. Would like to know if you get a wrong calculation or a exception?

@redquantum
Copy link
Author

I am sure it was doing wrong because it was working correctly on the binance exchange before, but now it is working incorrectly. When I started using the old version again, the error improved. In the old version, the Data matches the stock market every second.

I keep using the old version. We would appreciate if you publish the blocks of code changing in a file when the update is made.

Thank you very much..

@karlwancl
Copy link
Owner

@redquantum I should've updated the code in the commit 8565997 you can double check the block of code there.

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