Skip to content

Conversation

@nasa8x
Copy link

@nasa8x nasa8x commented Jun 1, 2025

Hi @BusinessDuck

I just added Ichimoku indicator, please check and modify it to fit your current library.

Thank you very much.

@BusinessDuck
Copy link
Contributor

BusinessDuck commented Jun 1, 2025

Good, thank you! MR looks good but we need to add:

  • XLS example of calculation
  • Unit test (cross sdk tests with technical indicators ichimoku)
  • Pass the review (i will do it by myself)

@BusinessDuck
Copy link
Contributor

@nasa8x

@nasa8x
Copy link
Author

nasa8x commented Jun 4, 2025

Sorry for the late reply. I tried but the cross-check between two libraries did not match. Please take the time to fix the error or rewrite.

Thank you.

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

Successfully merging this pull request may close these issues.

2 participants