-
-
Notifications
You must be signed in to change notification settings - Fork 359
FEATURE: [indicator] Add VWAP indicator #1258
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
base: main
Are you sure you want to change the base?
Conversation
…r-mm FEATURE: add risk controls for strategies
FIX: [grid2] use tryLock during the openGrid process
FEATURE: add v2 indicator set api
REFACTOR: [bollmaker] upgrade to indicator v2
…ders pkg/exchange: add QueryOpenOrders API for bybit
FEATURE: [bybit] support place order
FEATURE: [bybit] support cancel order
…-query-closed-order FIX: [max] return err on max queryClosedOrdersByLastOrderID
FEATURE: [bybit] query closed order
FEATURE: [max] add fee_discounted to Trade struct for RESTful api
…-end-time FIX: fix batch query trade missing time range
merge back v1.50 into main
|
Welcome back! @austin362667, This pull request may get 257 BBG. |
| return s | ||
| } | ||
|
|
||
| func (s *VWAPStream) calculate() float64 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to check both the rawCloseValues, rawVolumeValues length, because it could trigger twice since you have 2 update callbacks, or you can subscribe the KLine stream
|
|
v2