Skip to content

Removing OHLC_RSI for the moment, and focusing on a Random Trading St…#28

Merged
mccaffers merged 3 commits into
mainfrom
27-bug-ohlc_rsi-strategy-executes-0-trades-despite-processing-ticks
May 14, 2026
Merged

Removing OHLC_RSI for the moment, and focusing on a Random Trading St…#28
mccaffers merged 3 commits into
mainfrom
27-bug-ohlc_rsi-strategy-executes-0-trades-despite-processing-ticks

Conversation

@mccaffers
Copy link
Copy Markdown
Owner

@mccaffers mccaffers commented May 14, 2026 •
edited
Loading

No description provided.

@mccaffers mccaffers linked an issue May 14, 2026 that may be closed by this pull request
Bug: OHLC_RSI strategy executes 0 trades despite processing ticks #27
Closed
@mccaffers
Copy link
Copy Markdown
Owner Author

mccaffers commented May 14, 2026 

Executing query: SELECT 'EURUSD' as symbol, * FROM 'EURUSD' WHERE timestamp >= dateadd('M', -2, now()) ORDER BY timestamp
Total ticks streamed: 6895449
Final PnL: -379729.80
Trades opened: 169897  (LONG: 84999, SHORT: 84898)
Trades closed: 169896  (LONG: 84999, SHORT: 84897)
Winners: 28661   Losers: 140779   Breakeven: 456
Average PnL per closed trade: -2.24
Execution time: 5480ms

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 14, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
5.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@mccaffers mccaffers merged commit 8828128 into main May 14, 2026
2 of 3 checks passed
@mccaffers mccaffers deleted the 27-bug-ohlc_rsi-strategy-executes-0-trades-despite-processing-ticks branch May 14, 2026 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bug: OHLC_RSI strategy executes 0 trades despite processing ticks

1 participant

@mccaffers