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

FEATURE: [grid2] twin orderbook #1350

Merged
merged 3 commits into from Oct 24, 2023
Merged

FEATURE: [grid2] twin orderbook #1350

merged 3 commits into from Oct 24, 2023

Conversation

kbearXD
Copy link
Collaborator

@kbearXD kbearXD commented Oct 20, 2023

No description provided.

@kbearXD kbearXD requested a review from c9s October 20, 2023 08:27
@bbgokarma-bot
Copy link

Hi @kbearXD,

This pull request may get 403 BBG.

To receive BBG token, please left your polygon address as an issue comment in this pull request with the following format, e.g.,

polygon:0xAb5801a7D398351b8bE11C439e05C5B3259aeC9B

Once this pull request is merged, your BBG token will be sent to your wallet.

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 456 BBG

@kbearXD kbearXD requested a review from c9s October 23, 2023 05:02
@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

Merging #1350 (c977b8e) into main (f8c47f7) will increase coverage by 0.17%.
Report is 8 commits behind head on main.
The diff coverage is 54.90%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1350      +/-   ##
==========================================
+ Coverage   20.76%   20.93%   +0.17%     
==========================================
  Files         566      566              
  Lines       40357    40554     +197     
==========================================
+ Hits         8380     8491     +111     
- Misses      31368    31446      +78     
- Partials      609      617       +8     
Files Coverage Δ
pkg/strategy/grid2/twin_order.go 43.29% <54.90%> (+43.29%) ⬆️

... and 4 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f8c47f7...c977b8e. Read the comment docs.

@c9s c9s changed the title FEATURE: twin orderbook FEATURE: [grid2] twin orderbook Oct 23, 2023
@kbearXD
Copy link
Collaborator Author

kbearXD commented Oct 23, 2023

polygon:0x8642D208dce7214De9F222d864383968B5C346C7

@bbgokarma-bot
Copy link

Hi @kbearXD,

You left an invalid address format, please write your address with the following format:

polygon:0xAb5801a7D398351b8bE11C439e05C5B3259aeC9B

@bbgokarma-bot
Copy link

Great! @kbearXD, I've memorized your address.

// sort it in asc order
sort.Slice(v, func(i, j int) bool {
return v[j].Compare(v[i]) > 0
})
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pull this out to a small function?

for i, pin := range v {
m[pin] = &TwinOrder{}
pinIdx[pin] = i
}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this too?

}

if order.Side == types.SideTypeBuy {
idx++
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you please add a comment here? explain why idx needs to add 1

Repository owner deleted a comment from bbgokarma-bot Oct 23, 2023
Repository owner deleted a comment from bbgokarma-bot Oct 23, 2023
Repository owner deleted a comment from kbearXD Oct 23, 2023
@bbgokarma-bot
Copy link

Great! @kbearXD, I've memorized your address.

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 558 BBG

@kbearXD kbearXD merged commit 7d97f57 into main Oct 24, 2023
4 checks passed
@kbearXD kbearXD deleted the feature/grid2/twin-orderbook branch October 24, 2023 05:00
@bbgokarma-bot
Copy link

Hi @kbearXD,

Well done! 563 BBG has been sent to your polygon wallet. Please check the following tx:

https://polygonscan.com/tx/0x5d1b3b662667d39a33cd33ccab8306d4a469ac13004a6fa8a651562e0a1f0a34

Thank you for your contribution!

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

Successfully merging this pull request may close these issues.

None yet

3 participants