Skip to content

Coppock Curve Strategy added. Issue 344#388

Merged
cinar merged 2 commits into
cinar:masterfrom
whencanibe:coppock-curve-strategy
Jun 7, 2026
Merged

Coppock Curve Strategy added. Issue 344#388
cinar merged 2 commits into
cinar:masterfrom
whencanibe:coppock-curve-strategy

Conversation

@whencanibe
Copy link
Copy Markdown
Contributor

Describe Request

Implements the Coppock Curve Strategy as requested in issue #344.
The Coppock Curve indicator already exists in the momentum package. This PR adds the strategy layer on top of it.

Fixed #344

Change Type

New strategy.

Signed-off-by: Eonyak Cho <eonyakcho@gmail.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.96970% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 91.91%. Comparing base (607513d) to head (144fd8f).

Files with missing lines Patch % Lines
strategy/momentum/coppock_curve_strategy.go 96.87% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #388      +/-   ##
==========================================
+ Coverage   91.86%   91.91%   +0.05%     
==========================================
  Files         227      228       +1     
  Lines        7177     7210      +33     
==========================================
+ Hits         6593     6627      +34     
+ Misses        494      493       -1     
  Partials       90       90              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cinar
Copy link
Copy Markdown
Owner

cinar commented Jun 7, 2026

Thank you for contributing!

@cinar cinar merged commit 5c5bfbd into cinar:master Jun 7, 2026
2 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Coppock Curve Strategy

3 participants