contek-io / io.contek.invoker Goto Github PK

为啥文档中描述的submit方法呢

io.contek.invoker.bybitlinear.api.websocket.market.InstrumentInfoChannel.InstrumentInfo
fields
open_interest

and other volumetric values should be Double, otherwise you will see notional value change in one unit of base currency (1 Bitcoin for example), Contracts in linears are expressed in coin fractions (Satoshis for BTC)

For both Linear and Inverse prices prev_price_24h_e4 etc, should be in Double too, most of the contracts tick size is not equal to 1.

Thanks

The file .github/workflows/main.yml says, the project uses JDK 15.
The build.gradle file configures the sourceCompatibility and targetCompatibility to JDK 1.8.
Which version should be used to build the project?

I could build the project with JDK 17, but it caused a bunch of warnings.
I could also build the project with JDK 1.8, except for the module bybitlinear-api.

The JDK version should be aligned over all project files.

[com.squareup.okhttp3/okhttp "4.10.0"] -> [org.jetbrains.kotlin/kotlin-stdlib "1.6.20"] -> [org.jetbrains.kotlin/kotlin-stdlib-common "1.6.20"]

but also

[com.squareup.okhttp3/okhttp "4.10.0"] -> [com.squareup.okio/okio-jvm "3.0.0"] -> [org.jetbrains.kotlin/kotlin-stdlib-common "1.5.31"]

Solution:
Update https://github.com/square/okhttp once they release their new version

EDIT: They seem to intend to release 4.11 still: square/okhttp#7358

RateLimitRule need to have option to set rate limited per hour or per minute, Bitmex websocket api allows 30 conncetions per hour, Rate limiter limits 20 per minute

 RateLimiter createRateLimiter(String key, double cushion) {
    return RateLimiter.of(
        Joiner.on('_').join(type, name, key),
        RateLimiterConfig.custom()
            .limitForPeriod((int) (maxPermits * (1d - cushion)))
            .limitRefreshPeriod(Duration.of(1, MINUTES))
            .build());
  }

If IOException is common Okio keeps reconnecting more than 20 times per hour, which leads to
429 Too Many Requests' following '403 Forbidden' following permaban of IP

I guess simply changing

.limitRefreshPeriod(Duration.of(1, MINUTES))

with

 .limitRefreshPeriod(resetPeriod)

Will do the trick?

io.contek.invoker.bitmex.api.common._Instrument
missing few very important fields, without which is impossible to work with USDT instrument:

"underlyingToPositionMultiplier"
"underlyingToSettleMultiplier"
"quoteToSettleMultiplier"

api...

Thanks.

Some response models, e.g. for the Deribit API (package io.contek.invoker.deribit.api.common), use primitive datatypes like double, long, boolean, etc. instead of the classes Double, Long, Boolean, etc.

The problem with this is, that primitive datatypes in Java cannot be null. If a JSON response from an exchange doesn't contain a value, Gson sets default values. For numbers the default value is 0 (long, int) resp. 0.0 (double), for boolean it is false.

The actual problem with this behaviour is that after deserialization one cannot distinguish anymore if the exchange actually returned the value 0 or false, or if the response didn't contain the value.

The solution is simple, use classes instead of primitives. Classes can be checked for null.

This change also preserves backwards compatibility.

I'm willing to do the update and create a pull request, but I need some kind of a confirmation that this change is accepted, because it's plenty of work and I don't want to waste time if this change is not wanted.

Is this library still working? I was trying to launch example and noticed it uses wrong url for binance api request - I mean "dapi/v1/depth" - doesnt work but I can get right responce using api/v3/depth. Is it ok?

On Maven Central one can find v3.8.0 from Mar 2024 of parts of the modules (bitmex, bybit, deribit, okx). The source code of this version (and every other after 3.6.19) is not available here on this GitHub repository.

Is the source code now maintained somewhere else? Why is it not available here?

batch 下单和撤单能否加进去

Binance ws api ForceOrderEvent.Order both for linears and inverse has wrong field for liqudation price.
As per manual it should be public Double ap; instead of a.

I dont think its worth pr, so just reporting.

Hello!
Could anyone please add support for gate.io exchange?
I tried forking and copy/paste from binance and started to fill in for the gate.io api but it just got really complicated since I don't fully understand all the layers of the library.

I would be most interested in the WebSocket API that can be found here: https://www.gate.io/docs/developers/apiv4/ws/en/#api-overview

Or if someone can help me setup a new module for gateio spot api, then I could start filling in the blanks.
Thank you!

I'm not sure is it a bug or intention does this line needs break statement?

As described in the binance api doc, when symbol parameter provided the order book API will returns an object instead of an array:
https://binance-docs.github.io/apidocs/spot/en/#symbol-order-book-ticker

Therefore https://github.com/contek-io/io.contek.invoker/blob/master/binancespot-api/src/main/java/io/contek/invoker/binancespot/api/rest/market/GetTickerBookTicker.java will fail to execute, when user provide the Symbol parameter since Gson fail to convert the json string back to array.

Suggest fix would be separate the command into two where one for all symbols and one for single symbol...

Hello

1. The readme contains incorrect dependencies:

Correct is:

io.contek.invoker
invoker-deribit-api
2.15.0

2. When I add Deribit dependency - I find error:
   java: cannot access io.contek.invoker.security.ApiKey
   class file for io.contek.invoker.security.ApiKey not found

Hello,

I am starting to use this SDK to integrate ByBit APIs in my project. There is one small confusion, which one is the correct one to use :

1. Bybit-api
2. Bybitlinear-api

Thanks.

This library looks promising, but why do you use Double instead of BigDecimal? Any plans on using BigDecimal?

你好，能帮忙写个基于xchange的币安期货接口吗，那边只有现货的，有偿，微信676821842

Bybit has added a new API version V5, that unifies all old APIs (Spot, Derivatives, Options).

https://bybit-exchange.github.io/docs/v5/upgrade-guide

It's already helpful that the old derivatives API module "bybit-api" was renamed to "bybitinverse-api".

A new API module "bybit-api" could be added, implementing the Bybit V5 API. This preserves backwards compatibility for everyone using the existing modules.

These classes should be pulbic:
io.contek.invoker.binancelinear.api.websocket.market.combined.WebSocketCommand
io.contek.invoker.binancelinear.api.websocket.market.combined.WebSocketCommandConfirmation

Hello, recently ftx has change his rate limits, https://help.ftx.com/hc/en-us/articles/360052595091-2020-11-20-Ratelimit-Updates.

Should the API change for the case of FTX ?