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

Assertion failed: data must be asc ordered by time #47

Open
bludnic opened this issue Jul 27, 2024 · 1 comment
Open

Assertion failed: data must be asc ordered by time #47

bludnic opened this issue Jul 27, 2024 · 1 comment
Labels
bug Something isn't working ui

Comments

@bludnic
Copy link
Owner

bludnic commented Jul 27, 2024

Steps to reproduce:

  1. Create a bot (e.g. with RSI strategy) with 1m timeframe
  2. Open the bot page and start the bot
  3. Switch to 1m chart
  4. Disable internet connection at least for 2 minutes, to ensure we lost some candles
  5. Restore internet connection

image

Related to lightweight-charts tradingview/lightweight-charts#1263 (comment).

Most likely, some candles were lost due to an unstable WebSocket connection. When the connection is restored and a new candle is received, it does create a gap in the chart data.

@bludnic bludnic added bug Something isn't working ui labels Jul 27, 2024
@bludnic bludnic modified the milestone: v1.0.0-beta.0 Jul 28, 2024
@bludnic
Copy link
Owner Author

bludnic commented Dec 15, 2024

Similar issue when using Bitget 5ba2daa, which is partially fixed. Normally, we need to normalize the data (avoid duplicate candles or fix gaps) from the exchange, as CCXT doesn't handle this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ui
Projects
None yet
Development

No branches or pull requests

1 participant