Closed cinar closed 1 month ago
The pull request introduces significant enhancements to the Indicator Go module, particularly focusing on the Chaikin Money Flow strategy. Key updates include a complete rewrite of the README.md
to reflect version 2 features, the addition of the ChaikinMoneyFlowStrategy
type with associated methods, and improvements in the testing framework. The library now supports Golang generics and allows full configurability of indicators and strategies. Additionally, the MoneyFlowIndexStrategy
has minor formatting updates, and the AllStrategies
function now includes the new Chaikin strategy.
File | Change Summary |
---|---|
README.md |
Complete rewrite highlighting version 2 features, enhanced code quality, improved testability with CSV data, configurable indicators, and dual licensing details. |
strategy/volume/README.md |
Added ChaikinMoneyFlowStrategy type and methods; detailed strategy functionality and usage. |
strategy/volume/chaikin_money_flow_strategy.go |
Introduced ChaikinMoneyFlowStrategy struct, constructors, Compute , and Report methods for processing and reporting on asset snapshots. |
strategy/volume/chaikin_money_flow_strategy_test.go |
Added unit tests for the ChaikinMoneyFlowStrategy , validating computation and report generation. |
strategy/volume/money_flow_index_strategy.go |
Updated Name method for better formatting of SellAt and BuyAt values; clarified comment in Report method. |
strategy/volume/volume.go |
Added invocation of NewChaikinMoneyFlowStrategy() in AllStrategies function to include the new strategy. |
README.md
reflecting enhancements and features of version 2 of the Indicator Go module, similar to this PR's documentation updates.README.md
to include this new strategy, aligning with the focus on volume strategies.🐇 In the garden, I hop and play,
With strategies bright, we leap all day.
The Chaikin flows, with actions so clear,
In the world of finance, we have no fear!
With tests and reports, our work is done,
Hooray for the changes, let’s all have fun! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 96.29630%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 92.80%. Comparing base (
1cf5812
) to head (178b9af
).
Files with missing lines | Patch % | Lines |
---|---|---|
strategy/volume/chaikin_money_flow_strategy.go | 98.07% | 1 Missing :warning: |
strategy/volume/volume.go | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Describe Request
Chaikin Money Flow Strategy is added.
Change Type
New strategy.
Summary by CodeRabbit
New Features
Bug Fixes
Tests