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

TRXBTC symbol for 1M has negative volume #83

Open
sguryev opened this issue Apr 2, 2018 · 3 comments
Open

TRXBTC symbol for 1M has negative volume #83

sguryev opened this issue Apr 2, 2018 · 3 comments

Comments

@sguryev
Copy link

sguryev commented Apr 2, 2018

throw new ArgumentException($"{nameof(Candlestick)}: volume must not be less than 0.", nameof(volume));

Reprosteps: await _binanceApi.GetCandlesticksAsync("TRXBTC", CandlestickInterval.Month);
Json:
image

Inner exception: Candlestick: volume must not be less than 0. Parameter name: volume

StackTrace: at Binance.Candlestick..ctor(String symbol, CandlestickInterval interval, DateTime openTime, Decimal open, Decimal high, Decimal low, Decimal close, Decimal volume, DateTime closeTime, Decimal quoteAssetVolume, Int64 numberOfTrades, Decimal takerBuyBaseAssetVolume, Decimal takerBuyQuoteAssetVolume)
at Binance.Serialization.CandlestickSerializer.<>c__DisplayClass14_0.b__0(JToken item)
at System.Linq.Enumerable.SelectIListIterator2.ToArray() at System.Linq.Enumerable.ToArray[TSource](IEnumerable1 source)
at Binance.BinanceApi.d__26.MoveNext()

@sonvister
Copy link
Owner

@sguryev, thanks for reporting the exception. I have asked if the data is valid on the official API Telegram. Although, I can't imagine what a negative volume would represent if it is valid data.

@sguryev
Copy link
Author

sguryev commented Apr 2, 2018

@sonvister absolutely agree with you. I'm getting a bunch of symbols for 1M using Task.WhenAll() and now have to make a tricky Task.Run() workaround in order to swallow the exception. We can point them to the specific API call and ask about this negative value though

sonvister added a commit that referenced this issue Apr 4, 2018
@sonvister
Copy link
Owner

UPDATE: The query (https://api.binance.com/api/v1/klines?symbol=TRXBTC&interval=1M) still returns a negative volume....

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

No branches or pull requests

2 participants